Skip to content

Conversation

@alex-rakowski
Copy link
Collaborator

This PR changes the linter for flake8 to Ruff. We should agree on how exactly we want the linter configured before squashing and merging

  • adds ruff.toml to configure ruff
  • adds ruff.yml to configure and run ruff on only changed files in Github Actions
  • updates pre-commit.yml to run black and ruff and removes flake8
  • removes various flake8 config and Github Action files.

@alex-rakowski alex-rakowski requested a review from bsavitzky July 12, 2024 00:09
@CSSFrancis
Copy link

Just a comment, The Pre-commit CI is often a little easier to use then creating a github workflow

https://pre-commit.ci

I like it as you can just format other peoples code by running "pre-commit.ci autofix" which is helpful if you are trying to enforce things like code quality checks etc.

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