Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: switch from poetry to uv for environment management, and poetry to hatch for build backend #192

Merged
merged 3 commits into from
Jan 15, 2025

Conversation

drake-nominal
Copy link
Contributor

@drake-nominal drake-nominal commented Jan 15, 2025

UV offers numerous benefits over poetry, first and foremost with its speed. Without using poetry to manage the environment, it then made sense to switch to hatch for the build-backend, as it is a much more targeted project and excellently documented when it comes to writing the pyproject.toml.

I manually reviewed the generated wheels and sdists (.tar.gz) artifacts produced by uv build and poetry build, and the only difference I can detect is in the METADATA file created in the sdist, which is upgraded to a newer version with slightly different ways of presenting largely the same information.

A lot of the work ultimately came down to converting from the poetry-specific version specifier of ^x.y.z to the pep-compliant>=x.y.z,<x+1, as well as =* => >=0.0.0.

@drake-nominal drake-nominal force-pushed the deidukas/switch-to-uv branch 2 times, most recently from 7744ac7 to 63e405d Compare January 15, 2025 02:19
@drake-nominal drake-nominal marked this pull request as ready for review January 15, 2025 02:31
@drake-nominal drake-nominal requested review from alkasm, najork and jackparmer and removed request for alkasm January 15, 2025 02:31
pyproject.toml Show resolved Hide resolved
Copy link
Contributor

@alkasm alkasm left a comment

Choose a reason for hiding this comment

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

This looks great! Appreciate the thoroughness, stoked to bring in uv!

pyproject.toml Outdated Show resolved Hide resolved
@drake-nominal drake-nominal merged commit 0300415 into main Jan 15, 2025
5 checks passed
@drake-nominal drake-nominal deleted the deidukas/switch-to-uv branch January 15, 2025 17:51
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.

3 participants