Skip to content
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 info to setup.py so it will show up in wheel installs #137

Merged
merged 1 commit into from
Feb 1, 2022

Conversation

itsbenweeks
Copy link
Contributor

Just unable to see the license info from packaging. This will correct that.

Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @itsbenweeks, thanks a lot for your contribution! I left a small comment for you, otherwise looks good to me.

setup.py Outdated Show resolved Hide resolved
@ccordoba12
Copy link
Member

@itsbenweeks, do you need help to finish this one? We can give you hand so we can have this change in our next version.

@itsbenweeks
Copy link
Contributor Author

Sorry about the slow response. Got this committed and up to date, now.

@ccordoba12
Copy link
Member

Thanks @itsbenweeks! But since we moved our metadata from setup.py to setup.cfg, you need to do the same here.

Just take a look at PR #84 to see how to do that.

@itsbenweeks
Copy link
Contributor Author

Thanks for letting me know, @ccordoba12! It's been changed accordingly.

@ccordoba12 ccordoba12 added this to the v1.4.0 milestone Feb 1, 2022
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your help with this @itsbenweeks!

@ccordoba12 ccordoba12 changed the title Add license info to setup.py so it will show up in wheel installs. Add license info to setup.py so it will show up in wheel installs Feb 1, 2022
@ccordoba12 ccordoba12 merged commit aa6315d into python-lsp:develop Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants