Skip to content

fix: exclude sdk-core gitfile from wheels#1669

Open
Sakshamm-Goyal wants to merge 2 commits into
temporalio:mainfrom
Sakshamm-Goyal:fix/1659-wheel-gitfile
Open

fix: exclude sdk-core gitfile from wheels#1669
Sakshamm-Goyal wants to merge 2 commits into
temporalio:mainfrom
Sakshamm-Goyal:fix/1659-wheel-gitfile

Conversation

@Sakshamm-Goyal

@Sakshamm-Goyal Sakshamm-Goyal commented Jul 22, 2026

Copy link
Copy Markdown

Summary

  • exclude the initialized sdk-core submodule gitfile from Maturin wheel contents

Fixes #1659

Validation

  • uv run pytest -q tests/test_packaging.py
  • uv run ruff check tests/test_packaging.py
  • uv run ruff format --check tests/test_packaging.py pyproject.toml
  • uv build --wheel --out-dir /tmp/temporal-1659-after-20260722 followed by archive and RECORD inspection confirming temporalio/bridge/sdk-core/.git has zero entries
  • uv run python -c "import temporalio.bridge.temporal_sdk_bridge"

@Sakshamm-Goyal
Sakshamm-Goyal requested a review from a team as a code owner July 22, 2026 13:19
Comment thread tests/test_packaging.py Outdated
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.

[Bug] Published wheels contain stray submodule gitfile temporalio/bridge/sdk-core/.git

2 participants