Skip to content

fix: exclude sdk-core gitfile from wheels#1669

Merged
tconley1428 merged 2 commits into
temporalio:mainfrom
Sakshamm-Goyal:fix/1659-wheel-gitfile
Jul 23, 2026
Merged

fix: exclude sdk-core gitfile from wheels#1669
tconley1428 merged 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
Contributor

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
@tconley1428
tconley1428 merged commit f942f2c into temporalio:main Jul 23, 2026
30 of 31 checks passed
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