Skip to content

Switch from flake8 to ruff - #896

Merged
marcelm merged 15 commits into
mainfrom
ruff
Sep 14, 2026
Merged

marcelm merged 15 commits into
mainfrom
ruff

Conversation

@marcelm

@marcelm marcelm commented Sep 14, 2026

Copy link
Copy Markdown
Owner

And fix some of the things it complains about (mostly using ruff check --fix).

@rhpvorderman

Copy link
Copy Markdown
Collaborator

From the ruff page:

⚡️ 10-100x faster than existing linters (like Flake8) and formatters (like Black)

I love these people 😍 ! I will give this a look. I am subscribed to cutadapt in order to catch things when I mess up (python-isal issues usually resurface here first), and I am glad to catch PRs such as this where I can learn new things. So thanks for that.

@marcelm

marcelm commented Sep 14, 2026

Copy link
Copy Markdown
Owner Author

Yeah, it’s almost instant, and if there are no errors the first time you run it you wonder whether it did anything at all!

@marcelm
marcelm force-pushed the ruff branch 2 times, most recently from 79f4c76 to 4177d5e Compare September 14, 2026 14:25
@marcelm
marcelm merged commit a6e9385 into main Sep 14, 2026
25 checks passed
@marcelm
marcelm deleted the ruff branch September 14, 2026 15:03
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