Skip to content

feat(build-library): use uv build when use-uv is true#1268

Merged
SMoraisAnsys merged 4 commits intomainfrom
maint/uv-build
Apr 24, 2026
Merged

feat(build-library): use uv build when use-uv is true#1268
SMoraisAnsys merged 4 commits intomainfrom
maint/uv-build

Conversation

@viseshrp
Copy link
Copy Markdown
Contributor

@viseshrp viseshrp commented Apr 20, 2026

When use-uv: true (the default), the build step now uses uv build instead of python -m build, avoiding the need to install the build package as a separate dependency and also speeding up builds at the same time. The python -m build path is fully preserved when use-uv: false.

Changes:

Skip installing the build package when use-uv: true (only twine is needed for validation)
Run uv build --out-dir dist instead of python -m build --outdir dist when use-uv: true

@viseshrp viseshrp requested a review from a team as a code owner April 20, 2026 15:58
Copy link
Copy Markdown
Contributor

@klmcadams klmcadams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense to me, thanks!

Copy link
Copy Markdown
Contributor

@SMoraisAnsys SMoraisAnsys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, if work associated to astral-sh/uv#8641 get handled, we might even get rid of twine on this workflow

Copy link
Copy Markdown
Contributor

@MaxJPRey MaxJPRey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @viseshrp for the contribution.

@SMoraisAnsys SMoraisAnsys merged commit a86a121 into main Apr 24, 2026
81 of 83 checks passed
@SMoraisAnsys SMoraisAnsys deleted the maint/uv-build branch April 24, 2026 12:55
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.

5 participants