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
Adjust license for package uploaded to PyPI #1598
Adjust license for package uploaded to PyPI #1598
Conversation
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
d861dce
to
cf8a2d2
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1598 +/- ##
=======================================
Coverage 79.91% 79.91%
=======================================
Files 197 197
Lines 21731 21731
Branches 4339 4339
=======================================
Hits 17366 17366
Misses 2213 2213
Partials 2152 2152
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only have one concern regarding the "license_file" key.
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, given their classification system
@meshula Note that the license will be reported correctly. The note about the classifier is really just for classifiers. They are used to filter packages on PyPI. But our Apache 2.0 license will be reported correctly in the package Metadata and in the UI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for catching this!
…n#1598) * Adjust license for package uploaded to PyPI * Don't set license_file Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com> Signed-off-by: Michele Spina <michelespina96@gmail.com>
I noticed that our package in PyPI was still advertising the old license (Modified Apache 2.0). So this PR fixes that to make sure. Our next release will now correctly show Apache 2.0.