-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Plugin licence #38
Comments
Hello there, While trying to publish or update a plugin, we discover a little surprised the error message related to the deployment of this PR :
See upstream on qgis-plugin-ci project (I'm one of the maintener): opengisch/qgis-plugin-ci#255 I think that this kind of change, which breaks the plugins'publication flow, should be discussed before to be implemented (an issue from 1 person seems to be insufficient), announced to the community, a warning campaign should be run and a transitional phase should be implemented (warning for 6 months, then error). This has a direct impact on hundreds (thousands?) of plugin developers on a community project with several million end users. Reverting sounds maybe too rought so I suggest modyfing the behavior to lower the level and make it a simple warning and in the meanwhile starting a communication and preventive work upstream:
|
Trying to get some discussion and seing that another PR has been merged and deployed without a word, I've moved this topic on QGIS Dev mailing list: https://lists.osgeo.org/pipermail/qgis-developer/2023-November/066252.html |
This issue has been migrated from the old QGIS issue tracker.
Author
Paolo Cavallini (@pcav)
Description
In most plugins there is no separate licence file, and in many there is no licence whatsoever.
I think this should be a requirement for accepting a plugin on the repo, and should ask authors to fix this.
IMHO we should establish a standard way to add it (e.g. provide a LICENSE file and put license header to all sources, probably integrating in Plugin Builder).
Unsure whether this is the right place to place this ticket - feel free to move it elsewhere.
The text was updated successfully, but these errors were encountered: