Skip to content

chore: switch posit-bakery from poetry to uv#398

Merged
bschwedler merged 11 commits into
mainfrom
feature/use-uv
Mar 26, 2026
Merged

chore: switch posit-bakery from poetry to uv#398
bschwedler merged 11 commits into
mainfrom
feature/use-uv

Conversation

@ianpittwood
Copy link
Copy Markdown
Contributor

@ianpittwood ianpittwood commented Mar 25, 2026

Summary

  • Replaces poetry build system with hatchling + uv-dynamic-versioning (PEP 517 compliant, same Jinja2 version format via dunamai)
  • Migrates dev dependencies from [tool.poetry.group.dev.dependencies] to PEP 735 [dependency-groups]; replaces poetry.lock with uv.lock
  • Updates posit-bakery/justfile and CI (ci.yml) to use uv sync, uv run, uv build, and astral-sh/setup-uv@v5

Test Plan

  • 1168 tests passing locally with uv run pytest
  • just test passes via updated justfile
  • CI passes (test + release jobs)

🤖 Generated with Claude Code

@ianpittwood ianpittwood requested a review from bschwedler as a code owner March 25, 2026 16:59
ianpittwood and others added 7 commits March 25, 2026 10:00
- Replace build-system backend from poetry_dynamic_versioning to hatchling
- Add [tool.hatch.version] with uv-dynamic-versioning source
- Fix PEP 508 syntax: remove spaces before parentheses in dependencies
- Replace [tool.poetry.group.dev.dependencies] with PEP 735 [dependency-groups]
- Remove all [tool.poetry*] tables
- Add [tool.uv-dynamic-versioning] with format-jinja (without enable = true)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@bschwedler bschwedler left a comment

Choose a reason for hiding this comment

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

Thanks for doing this!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did you intend to commit the plan(s)?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Nope looks like the worktree skill did it! Interesting.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 25, 2026

Test Results

1 229 tests  ±0   1 229 ✅ ±0   13m 25s ⏱️ ±0s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 2171bd4. ± Comparison against base commit ea21761.

♻️ This comment has been updated with latest results.

@bschwedler bschwedler enabled auto-merge March 26, 2026 11:40
@bschwedler bschwedler added this pull request to the merge queue Mar 26, 2026
@bschwedler bschwedler removed this pull request from the merge queue due to a manual request Mar 26, 2026
- .claude/settings.json: poetry run -> uv run permission
- .gitignore: poetry.lock section -> uv.lock section
- justfile: error message now says uv tool install
- setup-bakery/action.yml: pipx install -> uv tool install,
  add astral-sh/setup-uv@v6 step since uv isn't pre-installed
  on GHA runners like pipx is
@bschwedler bschwedler enabled auto-merge March 26, 2026 12:14
@bschwedler bschwedler added this pull request to the merge queue Mar 26, 2026
Merged via the queue into main with commit 1969d25 Mar 26, 2026
16 checks passed
@bschwedler bschwedler deleted the feature/use-uv branch March 26, 2026 12:45
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