Skip to content

chore: Move ruff completely to uv#6430

Merged
sl0thentr0py merged 1 commit into
chore/migrate-to-tox-uvfrom
neel/chore/uv-dev
May 28, 2026
Merged

chore: Move ruff completely to uv#6430
sl0thentr0py merged 1 commit into
chore/migrate-to-tox-uvfrom
neel/chore/uv-dev

Conversation

@sl0thentr0py
Copy link
Copy Markdown
Member

@sl0thentr0py sl0thentr0py commented May 27, 2026

this will finally make pre-commit consistent with CI 🙃

@sl0thentr0py sl0thentr0py requested a review from a team as a code owner May 27, 2026 18:03
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2026

Codecov Results 📊

67 passed | Total: 67 | Pass Rate: 100% | Execution Time: 6.85s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 14803 uncovered lines.
❌ Project coverage is 35.08%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    35.14%    35.08%    -0.06%
==========================================
  Files          190       190         —
  Lines        22877     22803       -74
  Branches      7834      7810       -24
==========================================
+ Hits          8040      8000       -40
- Misses       14837     14803       -34
- Partials       712       712         —

Generated by Codecov Action

@sl0thentr0py sl0thentr0py changed the base branch from master to chore/migrate-to-tox-uv May 27, 2026 18:06
@sl0thentr0py sl0thentr0py changed the title neel/chore/uv dev chore: Move ruff completely to uv May 27, 2026
Copy link
Copy Markdown
Contributor

@sentrivana sentrivana left a comment

Choose a reason for hiding this comment

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

Might be AGENTS.md needs updating after this too

Copy link
Copy Markdown

@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.

Fix All in Cursor

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

Reviewed by Cursor Bugbot for commit e343398. Configure here.

# version (configured in pyproject.toml), ensuring consistent behavior.
linters: python3.14
mypy: python3.14
ruff: python3.14
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Template comment not updated, will revert on regeneration

Low Severity

The comment in tox.ini was updated to remove the ruff mention ("Tools like mypy"), but the source template tox.jinja still says "Tools like ruff and mypy". Since tox.ini is auto-generated from tox.jinja, the next regeneration will revert the tox.ini comment back to mentioning ruff, creating a stale/incorrect reference.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e343398. Configure here.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

will fix it later it's fine

@sl0thentr0py sl0thentr0py merged commit ad243c4 into chore/migrate-to-tox-uv May 28, 2026
157 of 159 checks passed
@sl0thentr0py sl0thentr0py deleted the neel/chore/uv-dev branch May 28, 2026 12:18
sl0thentr0py added a commit that referenced this pull request May 28, 2026
this will **finally** make pre-commit consistent with CI
:upside_down_face:
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