Skip to content

Conversation

@tstirrat15
Copy link
Contributor

@tstirrat15 tstirrat15 commented Sep 17, 2025

Fixes #283

Description

Poetry has consistently been a pain to use, and uv is a tool that seems to work fast and well. This migrates this project to using it.

Changes

  • Move all source code under a src folder
  • Update pyproject.toml to use new build system
  • Add lockfile
  • Remove old poetry configuration
  • Update actions accordingly
  • Update buf generation to dump into the src folder

Testing

See that all test steps still pass. I'll test the packaging by creating a prerelease and then testing it locally.

@tstirrat15 tstirrat15 requested a review from a team as a code owner September 17, 2025 17:18
@tstirrat15 tstirrat15 force-pushed the 283-use-uv-for-dep-management-and-building branch 3 times, most recently from 0055ad8 to 365febd Compare September 17, 2025 17:30
Copy link
Member

@jzelinskie jzelinskie left a comment

Choose a reason for hiding this comment

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

minor nitpick

pyproject.toml Outdated
Comment on lines 38 to 43
"authzed",
"buf",
"google",
"grpcutil",
"protoc_gen_openapiv2",
"validate",
Copy link
Member

Choose a reason for hiding this comment

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

this should probably be indented like the rest of the file

@tstirrat15 tstirrat15 force-pushed the 283-use-uv-for-dep-management-and-building branch from 365febd to 957ddf6 Compare September 17, 2025 17:48
@tstirrat15 tstirrat15 merged commit 5c45fc5 into main Sep 17, 2025
12 of 13 checks passed
@tstirrat15 tstirrat15 deleted the 283-use-uv-for-dep-management-and-building branch September 17, 2025 18:12
@github-actions github-actions bot locked and limited conversation to collaborators Sep 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move to uv for deps + building

3 participants