Skip to content

Conversation

@devinmatte
Copy link
Member

@devinmatte devinmatte commented Nov 25, 2025

Closes #155

Moves from poetry to uv, and adds extra checks to ensure we don't merge PRs that won't deploy going forward

Also includes the fix to actually allow deploys

@devinmatte devinmatte marked this pull request as ready for review November 25, 2025 16:51
@devinmatte devinmatte requested a review from ankoure December 5, 2025 18:39
@devinmatte devinmatte changed the title Migrate to UV and improve checks Migrate to UV, fix delay processing, and improve checks Dec 5, 2025
Copy link
Member

@nathan-weinberg nathan-weinberg left a comment

Choose a reason for hiding this comment

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

One nit but LGTM

description = ""
authors = ["TransitMatters Labs Team"]
authors = [{ name = "TransitMatters Labs Team" }]
requires-python = ">=3.13,<3.14"
Copy link
Member

Choose a reason for hiding this comment

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

Why not just pin to 3.13?

Copy link
Member Author

Choose a reason for hiding this comment

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

idk this is what UV did automatically in the migration

@devinmatte devinmatte merged commit dd79a76 into main Dec 6, 2025
3 checks passed
@devinmatte devinmatte deleted the uv-testing branch December 6, 2025 21:30
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.

Migrate from Poetry to UV

3 participants