Skip to content

Conversation

@leifdreizler
Copy link
Contributor

This changes the license field to be a valid SPDX identifier aligning with PEP 639. This populates the license_expression field in the PyPI API and is used by downstream tools including deps.dev

These changes were generated by Claude (sorry it wasn't Gemini) after reviewing the license and manifest files in your repository, but opened and reviewed by me. Please let me know if the analysis is incorrect and thanks for being an OSS maintainer.

This changes the license field to be a valid [SPDX identifier](https://spdx.org/licenses) aligning with [PEP 639](https://peps.python.org/pep-0639/#project-source-metadata). This populates the `license_expression` field in the PyPI API and is used by downstream tools including deps.dev

These changes were generated by Claude after reviewing the license and manifest files in your repository, but opened and reviewed by me. Please let me know if the analysis is incorrect and thanks for being an OSS maintainer.
@barbibulle
Copy link
Collaborator

Thanks for the PR. Shouldn’t pyproject.toml also include a license-files entry as well, pointing to `LICENSE?

@barbibulle barbibulle self-requested a review August 2, 2025 05:10
@leifdreizler
Copy link
Contributor Author

Yes, happy to make that change! ✅

I have only been making that when folks had the older license = {file = "LICENSE"} file sub key to minimize my changes, but perhaps I should just add it 🤔

@barbibulle barbibulle merged commit efdc770 into google:main Aug 2, 2025
57 checks passed
@leifdreizler leifdreizler deleted the fix-spdx-license branch August 2, 2025 18:29
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