-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Add linting back to CI #3622
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
Comments
I like the idea and have heard many good things about |
Tracking PRs: Formatting
Linting
|
Hello @Viicos, @behackl and @JasonGrace2282, how are you? I was analyzing this issue and I found it very interesting! Maybe I could help with one of the following linting topics:
I´m still analyzing the possibility of acting on this issue and as soon as I decide I will let you know. Thanks! |
Hello @Viicos, @behackl and @JasonGrace2282, how are you? As I mentioned in the above comment, I will help with this issue by working on:
I will implement Ruff linting CI to replace pylint and once I finished the code, I will create the PR. Thanks, guys! |
Following #3620, would be great to enforce linting on CI.
I propose using Ruff instead of flake8, it would avoid having to install a bunch of plugins and
pyproject.toml
configuration is supported.Format could be included as well if the diff isn't to big.
The text was updated successfully, but these errors were encountered: