Skip to content

uv#58

Merged
pda merged 5 commits intomainfrom
uv
Jul 18, 2025
Merged

uv#58
pda merged 5 commits intomainfrom
uv

Conversation

@pda
Copy link
Member

@pda pda commented Jul 16, 2025

The first/main commit on this branch is the unmodified result of prompting Claude Sonnet 4:

Migrate this Python codebase to uv (https://docs.astral.sh/uv)

I opened it as a draft pull request to invite review (from myself and others) etc before merging.

I've since made some adjustments, and done some other work on branch off this one using the uv tooling, and it seems good. The GitHub Actions checks which were updated in this PS continue to work correctly, the tests are running and passing.

This patch is the result of prompting Claude Sonnet 4:

> Migrate this Python codebase to uv (https://docs.astral.sh/uv)
@pda pda requested a review from a team July 16, 2025 07:56
3. Set up your API token

Add the `BUIDLKITE_ANALYTICS_TOKEN` environment variable to your build system's environment.
Add the `BUILDKITE_ANALYTICS_TOKEN` environment variable to your build system's environment.
Copy link
Contributor

Choose a reason for hiding this comment

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

🦅 👁️

@pda pda requested a review from a team July 17, 2025 11:08
@pda
Copy link
Member Author

pda commented Jul 17, 2025

I've made some minor adjustments, but basically I think this is good to ship.
I don't think it changes anything for users/consumers of the code, only developers of the plugin.

If there's any new friction in the release process, we can update that process when the time comes, as we always do 😅

@pda pda marked this pull request as ready for review July 17, 2025 11:10
@pda pda merged commit ca12781 into main Jul 18, 2025
11 checks passed
@pda pda deleted the uv branch July 18, 2025 02:09
@gchan gchan mentioned this pull request Jul 21, 2025
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