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

Adjusting license tag to use SPDX identifier #345

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hyandell
Copy link

Hi Pyvisa folk,

My understanding is that, when possible, the license field ought to contain a descriptive identifier rather than link to the license. Typically that's when an SPDX license identifier exists. That makes it easier for automation, like the pypi website, to more cleanly describe the license.

As your license is the MIT license, here's a proposed PR to adjust it to be "MIT" rather than copying the whole license in.

@MatthieuDartiailh
Copy link
Member

Your approach aligns with PEP 639 but it has not been accepted yet. Do you know if it is already supported by setuptools ?

@hyandell
Copy link
Author

Thanks for the heads up on PEP 639; I come to Python by way of licensing in the Java world so hadn't discovered that yet.

I haven't seen that show up in any PyPI packages, so I'm guessing it's not supported yet, or not known.

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