Skip to content

chore(packaging): use package data instead of MANIFEST.in#18537

Merged
nijel merged 1 commit intoWeblateOrg:mainfrom
nijel:manifest
Mar 19, 2026
Merged

chore(packaging): use package data instead of MANIFEST.in#18537
nijel merged 1 commit intoWeblateOrg:mainfrom
nijel:manifest

Conversation

@nijel
Copy link
Member

@nijel nijel commented Mar 19, 2026

This better defines the purpose of data.

This better defines the purpose of data.
@nijel nijel added this to the 5.17 milestone Mar 19, 2026
@nijel nijel requested a review from Copilot March 19, 2026 11:50
@nijel nijel self-assigned this Mar 19, 2026
Copy link
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

This PR updates Weblate’s packaging configuration to declare Django-related assets (locales, static files, templates) via setuptools package-data rather than enumerating them in MANIFEST.in, improving clarity around what constitutes package data.

Changes:

  • Add [tool.setuptools.package-data] entries to include locale catalogs, static assets, and templates in the relevant packages.
  • Remove now-redundant recursive-include rules for those assets from MANIFEST.in.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pyproject.toml Defines package data globs for weblate and selected subpackages to include locales/static/templates via setuptools config.
MANIFEST.in Drops explicit recursive includes for assets now covered by setuptools package-data, keeping other sdist inclusions intact.

@nijel nijel enabled auto-merge (rebase) March 19, 2026 11:56
@nijel nijel disabled auto-merge March 19, 2026 12:26
@nijel nijel enabled auto-merge (squash) March 19, 2026 12:26
@nijel nijel merged commit a99282f into WeblateOrg:main Mar 19, 2026
53 checks passed
@nijel nijel deleted the manifest branch March 19, 2026 12:46
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