Skip to content

Comments

Move to ruff for linting#45

Merged
c-randall merged 3 commits intomainfrom
chore/ruff
Feb 16, 2026
Merged

Move to ruff for linting#45
c-randall merged 3 commits intomainfrom
chore/ruff

Conversation

@c-randall
Copy link
Collaborator

Description

Move to using ruff for linting and formatting instead of flake8 and autopep8. Also, currently following an issue requesting linting for Cython in ruff here, which will be beneficial to have some day.

Type of change

  • New feature (non-breaking change which adds functionality)
  • Optimization (back-end change that improves speed/readability/etc.)
  • Bug fix (non-breaking change which fixes an issue)
  • Chore (any other clean up, maintenance, etc.)

Key checklist

  • No style issues: $ nox -s linter [-- format]
  • Code is free of misspellings: $ nox -s codespell [-- write]
  • All tests pass: $ nox -s tests
  • Badges are updated: $ nox -s badges

Further checks:

  • The documentation builds: $ nox -s docs.
  • Code is commented, particularly in hard-to-understand areas.
  • Tests are added that prove fix is effective or that feature works.

@c-randall c-randall marked this pull request as ready for review February 16, 2026 18:27
@c-randall c-randall merged commit 04df826 into main Feb 16, 2026
19 checks passed
@c-randall c-randall deleted the chore/ruff branch February 16, 2026 18:29
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