Skip to content

Add ruff linting to Makefile format target#548

Closed
MaxGhenis wants to merge 1 commit intomasterfrom
add-ruff-linting
Closed

Add ruff linting to Makefile format target#548
MaxGhenis wants to merge 1 commit intomasterfrom
add-ruff-linting

Conversation

@MaxGhenis
Copy link
Contributor

Summary

  • Adds [tool.ruff.lint] config to pyproject.toml with E (pycodestyle), F (pyflakes), and I (isort) rule sets
  • Adds ruff check . --fix to the make format target (runs before ruff format .)
  • Auto-fixed import sorting across the codebase
  • Rules that conflict with existing code patterns (star imports, unused imports as re-exports, bare excepts, etc.) are ignored for now — these can be gradually enabled

This replaces flake8/pylint with ruff check. It is only added to the Makefile, not to CI, so it will not block any PRs.

Test plan

  • ruff check . passes with zero errors
  • No functional code changes — only import ordering

🤖 Generated with Claude Code

Configure ruff check with E (pycodestyle), F (pyflakes), and I (isort)
rule sets. Auto-fixed import sorting; remaining rules that conflict with
existing code patterns are ignored for now. This makes ruff check
available via `make format` without enforcing it in CI yet.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MaxGhenis
Copy link
Contributor Author

Import reordering caused circular imports. Needs more careful approach.

@MaxGhenis MaxGhenis closed this Mar 6, 2026
@MaxGhenis MaxGhenis deleted the add-ruff-linting branch March 6, 2026 14:02
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.

1 participant