Skip to content

Use --no-build-isolation for uv build after sync#39

Closed
Owen-Liuyuxuan wants to merge 1 commit into
mainfrom
fix/uv-build-no-isolation
Closed

Use --no-build-isolation for uv build after sync#39
Owen-Liuyuxuan wants to merge 1 commit into
mainfrom
fix/uv-build-no-isolation

Conversation

@Owen-Liuyuxuan
Copy link
Copy Markdown
Contributor

Summary

  • CI, docs deploy, and Cloudflare Pages builds now run uv build with --no-build-isolation so Hatch protobuf codegen uses the locked .venv from uv sync instead of re-resolving newer protobuf / mypy-protobuf in an isolated PEP 517 env.
  • Production build docs now require uv sync --all-packages before uv build --all-packages --no-build-isolation.

Test plan

  • uv sync --all-extras --dev --all-packages
  • Baseline: isolated uv build --package sync-tooling-msgs selected protobuf 6.33.6 / mypy-protobuf 5.1.0 (venv had 6.33.4)
  • After fix: uv build --package sync-tooling-msgs --no-build-isolation uses venv (no re-resolve Selecting lines)
  • uv build --package sync-tooling-msgs --no-build-isolation -o /tmp/dist-test succeeds
  • uv build --all-packages --no-build-isolation succeeds
  • CI workflow on this PR

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 2, 2026

@Owen-Liuyuxuan
Copy link
Copy Markdown
Contributor Author

Superseded by ground-up fix: unify protobuf codegen with uv.lock at uv sync via build-constraint-dependencies (PR #38 rework).

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.

1 participant