Skip to content

fix: migrates us fully to UV, support for PIP still maintained#2577

Merged
AidaPaul merged 6 commits intomainfrom
fix/complete_uv_migration
Feb 12, 2026
Merged

fix: migrates us fully to UV, support for PIP still maintained#2577
AidaPaul merged 6 commits intomainfrom
fix/complete_uv_migration

Conversation

@AidaPaul
Copy link
Contributor

@AidaPaul AidaPaul commented Feb 9, 2026

Description

Completes UV migration, still need to do the same for other repos and confirm across the entire stack

Checklist

  • I verified this PR does not break soda-extensions.

@AidaPaul AidaPaul force-pushed the fix/complete_uv_migration branch 4 times, most recently from c346b08 to 11748f6 Compare February 11, 2026 09:48
@AidaPaul AidaPaul marked this pull request as ready for review February 11, 2026 09:59
@AidaPaul AidaPaul requested review from m1n0, nielsn and paulteehan and removed request for m1n0 and paulteehan February 11, 2026 10:15
@AidaPaul AidaPaul force-pushed the fix/complete_uv_migration branch from 11748f6 to d6e0bc1 Compare February 11, 2026 14:44
@AidaPaul AidaPaul force-pushed the fix/complete_uv_migration branch from d6e0bc1 to 178a5eb Compare February 11, 2026 15:07
@sonarqubecloud
Copy link

Copy link
Contributor

@m1n0 m1n0 left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link
Contributor

@paulteehan paulteehan left a comment

Choose a reason for hiding this comment

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

Just tested scripts/recreate_venv.sh on a clean environment and it worked beautifully.

The rest I just skimmed, everything looks solid to me, my one concern would be validating that the CI script works, if you've tested that then okay to merge for me.

@AidaPaul
Copy link
Contributor Author

Just tested scripts/recreate_venv.sh on a clean environment and it worked beautifully.

The rest I just skimmed, everything looks solid to me, my one concern would be validating that the CI script works, if you've tested that then okay to merge for me.

I ran CI across it, seems to work so far!

@AidaPaul AidaPaul merged commit 4518065 into main Feb 12, 2026
41 checks passed
@AidaPaul AidaPaul deleted the fix/complete_uv_migration branch February 12, 2026 13:47
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