Skip to content

Migrate linting from flake8+isort to ruff#223

Merged
GlassOfWhiskey merged 1 commit into
masterfrom
migrate-to-ruff
May 3, 2026
Merged

Migrate linting from flake8+isort to ruff#223
GlassOfWhiskey merged 1 commit into
masterfrom
migrate-to-ruff

Conversation

@GlassOfWhiskey
Copy link
Copy Markdown
Member

Replace flake8, flake8-bugbear, and isort with ruff (rules E, W, F, B, I). black remains the sole formatter; ruff format is not used.

  • Delete .flake8; add [tool.ruff] sections to pyproject.toml
  • Update Makefile: fold flake8 target into format/format-check
  • Update tox.ini
  • Fix pre-existing violations surfaced by ruff: B905 (add strict=True to zip() calls)
  • Suppress B904 globally

Replace `flake8`, `flake8-bugbear`, and `isort` with `ruff` (rules
`E`, `W`, `F`, `B`, `I`). `black` remains the sole formatter;
`ruff format` is not used.

- Delete `.flake8`; add `[tool.ruff]` sections to `pyproject.toml`
- Update `Makefile`: fold `flake8` target into `format`/`format-check`
- Update `tox.ini`
- Fix pre-existing violations surfaced by `ruff`: `B905`
  (add `strict=True` to `zip()` calls)
- Suppress `B904` globally
@GlassOfWhiskey GlassOfWhiskey requested a review from LanderOtto May 2, 2026 20:03
@codecov
Copy link
Copy Markdown

codecov Bot commented May 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.13%. Comparing base (1729159) to head (a723e80).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #223   +/-   ##
=======================================
  Coverage   93.13%   93.13%           
=======================================
  Files           8        8           
  Lines         612      612           
=======================================
  Hits          570      570           
  Misses         42       42           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@GlassOfWhiskey GlassOfWhiskey merged commit f98bf2b into master May 3, 2026
13 checks passed
@GlassOfWhiskey GlassOfWhiskey deleted the migrate-to-ruff branch May 3, 2026 06:34
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