Use PEP 639 license information for Ruff itself instead of classifier#19499
Use PEP 639 license information for Ruff itself instead of classifier#19499konstin merged 1 commit intoastral-sh:mainfrom
Conversation
Declare licenses using only these two fields, as per PEP 639: * license: SPDX license expression consisting of one or more license identifiers * license-files: list of license file glob patterns Supported by maturin ≥ 1.9.0: https://www.maturin.rs/changelog.html
| "Development Status :: 5 - Production/Stable", | ||
| "Environment :: Console", | ||
| "Intended Audience :: Developers", | ||
| "License :: OSI Approved :: MIT License", |
There was a problem hiding this comment.
Because PEP639 expects only licence and licence-files.
It's redundant with:
license = "MIT"
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
Sorry, that PEP does indeed recommend removing the license classifier: https://peps.python.org/pep-0639/#deprecate-license-classifiers — I hadn’t looked at it carefully before.
konstin
left a comment
konstin
left a comment
There was a problem hiding this comment.
I confirmed that after this change, there's
License-File: LICENSE
License-Expression: MIT
in the METADATA and a license file at /ruff-0.12.4.dist-info/licenses/LICENSE
konstin
changed the title
konstin
added
documentation
|
I removed the documentation label so that it appears under the "Other" section (we tend to remove the Documentation section in releases) |
|
I see this PR has been fully reverted:
Will you @DimitriPapadopoulos have time to look into it? |
|
Unfortunately #19624 lacks context. Is the problem that |
5 participants
Summary
Declare licenses using only these two fields, as per PEP 639:
license: SPDX license expression consisting of one or more license identifierslicense-files: list of license file glob patternsSupported by maturin ≥ 1.9.0:
https://www.maturin.rs/changelog.html
Test Plan
N/A