-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Add license files consistently for all packages #19240
Conversation
Thank you for your contribution major! We will review the pull request and get back to you soon. |
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.
Attestation looks fine. Thank you very much.
Thanks! This helps a bunch for Fedora RPM packaging! 🎉 |
Ensure each python package uploaded to pypi contains a `LICENSE.txt` file along with an `include` line in the `MANIFEST.in`. Related to Azure/azure-cli#17909 Signed-off-by: Major Hayden <major@mhtx.net>
I just rebased this PR since it was pretty far behind the |
@lmazuel Hello! Scott recommended checking in with you to see if this change is reasonable or if I should break this up into multiple pieces. I'm happy to help either way. |
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.
Attestation looks fine.
Closing in favor of #20193 instead. |
Ensure each python package uploaded to pypi contains a
LICENSE.txt
file along with an
include
line in theMANIFEST.in
.Related to Azure/azure-cli#17909