Skip to content

Switch from poetry to uv #2049

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

Merged
merged 16 commits into from
Jul 5, 2025
Merged

Switch from poetry to uv #2049

merged 16 commits into from
Jul 5, 2025

Conversation

gantoine
Copy link
Member

@gantoine gantoine commented Jul 4, 2025

Description
Explain the changes or enhancements you are proposing with this pull request.

This PR migrates the project from poetry to uv, a new tool that bypasses the need to manage python versions and installs with pip, pipx, poetry, pyenv or virtualenv. Migrating pyproject.toml was automatic with migrate-to-uv, and in most cases calls to poetry were directly replaced. Setup docs have been updated to reflect the fact that python versions no longer need to be pre-installed/managed, and that uv takes care of everything.

Checklist
Please check all that apply.

  • I've tested the changes locally
  • I've updated relevant comments
  • I've assigned reviewers for this PR
  • I've added unit tests that cover the changes

@gantoine gantoine changed the base branch from release to master July 4, 2025 13:55
Copy link

trunk-io bot commented Jul 4, 2025

Running Code Quality on PRs by uploading data to Trunk will soon be removed. You can still run checks on your PRs using trunk-action - see the migration guide for more information.

Copy link

github-actions bot commented Jul 4, 2025

Test Results

91 tests  ±0   91 ✅ ±0   29s ⏱️ -2s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit a34686c. ± Comparison against base commit de762aa.

♻️ This comment has been updated with latest results.

@gantoine gantoine requested review from Copilot and adamantike and removed request for Copilot July 4, 2025 14:03
@gantoine gantoine marked this pull request as ready for review July 4, 2025 14:07
Copy link
Contributor

@adamantike adamantike left a comment

Choose a reason for hiding this comment

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

🚀

Comment on lines +93 to +97
````sh
uv venv
source .venv/bin/activate
uv sync --all-extras --dev
``
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
````sh
uv venv
source .venv/bin/activate
uv sync --all-extras --dev
``
```sh
uv sync --all-extras
```

Copy link
Member Author

Choose a reason for hiding this comment

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

I think for dev setup we should push ppl to create a venv and source it, for cleanliness

gantoine and others added 11 commits July 4, 2025 21:22
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
Co-authored-by: Michael Manganiello <adamantike@users.noreply.github.com>
@gantoine gantoine merged commit edcd82e into master Jul 5, 2025
9 checks passed
@gantoine gantoine deleted the poetry-to-uv branch July 5, 2025 02:02
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.

2 participants