-
Notifications
You must be signed in to change notification settings - Fork 223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replaced flake8 with ruff and added it to CI #306
Conversation
This looks OK but I'd like to keep the same rules (longer lines mainly). I have to try it locally and see, haven't used ruff yet but I hear good things. Might take me a couple of weeks to get back to this. |
No worries. Do you have any concrete suggestions I can already add so you are happy with the PR itself? 🙂 |
python-version: "3.12" | ||
|
||
- name: Install required packages | ||
run: pip install ruff |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be in poetry's dev dependencies, and get run on commit? Maybe we can add a pre-commit hook if one doesn't exist already.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pre-commit sounds very good. I'll take care of it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pushed new changes 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Finally got to testing this, thanks for the PR!
@pcraciunoiu Awesome, thx! Had totally forgotten about this one 😅 |
Hi @pcraciunoiu
I realised that flake8 is not run in the CI. Since it's kinda deprecated and Ruff is the new cool kid on the block, I created a PR. Ruff has so many nice add-ons but I didn't want to bloat this PR, so I just used it as a drop-in replacement.
What do you think?
Best from Cologne
Ronny