Skip to content

Restore LICENSE in sdist via PEP 639 license-files#9341

Merged
manzt merged 1 commit into
mainfrom
manzt/license
Apr 23, 2026
Merged

Restore LICENSE in sdist via PEP 639 license-files#9341
manzt merged 1 commit into
mainfrom
manzt/license

Conversation

@manzt
Copy link
Copy Markdown
Collaborator

@manzt manzt commented Apr 23, 2026

Fixes #9339

#9231 moved to the SPDX license = "Apache-2.0" expression but didn't add license-files, and uv_build doesn't glob LICENSE* by default, so the 0.23.2 sdist shipped without one.

Fixes #9339

#9231 moved to the SPDX `license = "Apache-2.0"` expression but didn't
add `license-files`, and uv_build doesn't glob `LICENSE*` by default, so
the 0.23.2 sdist shipped without one.
Copilot AI review requested due to automatic review settings April 23, 2026 15:41
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Apr 23, 2026 3:44pm

Request Review

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Restores the missing Apache 2.0 license file in the Python sdist by explicitly declaring it via PEP 639 metadata, addressing the regression introduced when switching to SPDX license = "Apache-2.0" without accompanying license-files.

Changes:

  • Add license-files = ["LICENSE"] to the [project] metadata so build tooling includes the license file in sdists.

@manzt manzt merged commit c445386 into main Apr 23, 2026
52 of 54 checks passed
@manzt manzt deleted the manzt/license branch April 23, 2026 20:10
@github-actions
Copy link
Copy Markdown

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.23.3-dev35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

License file missing in v0.23.2 sdist on PyPI

3 participants