Skip to content

feat: replace tox with uv #225

Open
joshuarli wants to merge 3 commits intomainfrom
replace-tox-with-uv
Open

feat: replace tox with uv #225
joshuarli wants to merge 3 commits intomainfrom
replace-tox-with-uv

Conversation

@joshuarli
Copy link
Member

much easier for ai to reason about requirements consolidated in uv + faster tools (also replaced black+flake8+reorderpythonimports with ruff and made pre-commit hooks local to be faster)

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@joshuarli joshuarli requested a review from a team March 3, 2026 18:13
Copy link

@bc-sentry bc-sentry left a comment

Choose a reason for hiding this comment

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

I think it would be easier to review if you had a separate PRs for the tox -> uv and the other changes ( black+flake8+reorderpythonimports with ruff and made pre-commit hooks local to be faster)

@joshuarli
Copy link
Member Author

I think it would be easier to review if you had a separate PRs for the tox -> uv and the other changes ( black+flake8+reorderpythonimports with ruff and made pre-commit hooks local to be faster)

for the non-formatting changes you can look at d998f61

@joshuarli joshuarli requested a review from bc-sentry March 3, 2026 19:05
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