Repository navigation
Switch from Poetry to uv - #378
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #378 +/- ##
=======================================
Coverage 92.18% 92.18%
=======================================
Files 398 398
Lines 25267 25267
=======================================
Hits 23293 23293
Misses 1974 1974 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
mshannon-sil
left a comment
mshannon-sil
left a comment
There was a problem hiding this comment.
@mshannon-sil made 1 comment.
Reviewable status: 0 of 14 files reviewed, all discussions resolved.
Enkidu93
left a comment
Enkidu93
left a comment
There was a problem hiding this comment.
@Enkidu93 reviewed 14 files and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on ddaspit).
|
Since the transformers PR is through, can this be rebased and merged now? |
|
@ddaspit, were you waiting on something in particular before rebasing and merging this? |
|
Nope, I just totally forgot about this. I will resurrect this PR. |
- Convert Poetry dependency groups to PEP 735 dependency groups and replace poetry.lock with uv.lock - Use uv_build as the build backend; include LICENSE in the distributions and publish the repository URL via [project.urls] - Keep installing the gpu group by default (default-groups = "all") to match Poetry's behavior - Update CI, dockerfiles, dev container, VS Code settings, local_check.sh and the port-pr skill Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
setup-uv does not publish floating major-version tags. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The Eflomal aligner is provided by Thot (sil-thot), so the eflomal package and EFLOMAL_PATH are no longer used. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
No other distribution contributes to the machine namespace, so it no longer needs to be a namespace package. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The Claude, Claude code review and comment hygiene workflows added on main still installed Poetry and keyed their caches on poetry.lock, which this branch removes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Replaces Poetry with uv for dependency management, builds, CI, Docker and the dev container.
poetry.lock→uv.lock(fresh lock, mostly patch-level bumps); Poetry groups → PEP 735 dependency groups,gpustill installed by defaultuv_build; the wheel now includes LICENSE and the repository URLmachinea regular package (adds__init__.py) instead of a namespace packageeflomalpackage andEFLOMAL_PATH(the Eflomal aligner comes from Thot)Tested: full CI matrix; the GPU Docker image builds and passes the Thot/jobs tests inside the container. Not tested:
dockerfile.cpu_only, the dev container, anduv publish(first runs on the next tag).🤖 Generated with Claude Code
This change is