Skip to content

Exclude most AsyncTI4 submodule content from sdist while keeping required data subtree - #39

Merged
adam133 merged 2 commits into
mainfrom
copilot/fix-package-size-issue
Apr 17, 2026
Merged

Exclude most AsyncTI4 submodule content from sdist while keeping required data subtree#39
adam133 merged 2 commits into
mainfrom
copilot/fix-package-size-issue

Conversation

Copilot AI commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

PyPI publish artifacts were oversized because the source distribution was bundling the full data/TI4_map_generator_bot git submodule. This change now constrains sdist contents to keep artifacts small while still including the required subdirectory from the submodule.

  • Packaging scope correction

    • Switched from a broad exclusion-only approach to an explicit sdist include set.
    • Kept wheel packaging behavior unchanged (src/ti4_rules_engine remains the wheel package root).
  • Build configuration update

    • Updated pyproject.toml with a dedicated sdist target configuration that includes:
      • src/ti4_rules_engine
      • data/TI4_map_generator_bot/src/main/resources/data
      • README.md
      • LICENSE
      • pyproject.toml
[tool.hatch.build.targets.sdist]
only-include = [
    "src/ti4_rules_engine",
    "data/TI4_map_generator_bot/src/main/resources/data",
    "README.md",
    "LICENSE",
    "pyproject.toml",
]

Agent-Logs-Url: https://github.com/adam133/ti4-rules-engine/sessions/ad8b046f-73c1-4ead-8eda-ff390ab18eb9

Co-authored-by: adam133 <20442729+adam133@users.noreply.github.com>
@adam133

adam133 commented Apr 17, 2026

Copy link
Copy Markdown
Owner

@copilot but we still need a subdirectory of the submodule included in the build.

Copilot AI commented Apr 17, 2026

Copy link
Copy Markdown
Contributor Author

@copilot but we still need a subdirectory of the submodule included in the build.

Updated the packaging config to include only the required subdirectory from the submodule (data/TI4_map_generator_bot/src/main/resources/data) while keeping the rest excluded from sdist. Fixed in commit 9f8df48.

Copilot AI changed the title Exclude AsyncTI4 submodule from sdist packaging Exclude most AsyncTI4 submodule content from sdist while keeping required data subtree Apr 17, 2026
Copilot AI requested a review from adam133 April 17, 2026 02:16
@adam133
adam133 marked this pull request as ready for review April 17, 2026 02:33
@adam133
adam133 merged commit 0e91d29 into main Apr 17, 2026
1 check failed
@adam133
adam133 deleted the copilot/fix-package-size-issue branch April 17, 2026 02:34
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