-
-
Notifications
You must be signed in to change notification settings - Fork 284
Adding supported filetypes to ruff and ruff-format linters #6186
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
Conversation
229c2f6 to
ace23fd
Compare
|
If you're willing, in a second PR, black also can do the same: https://black.readthedocs.io/en/stable/usage_and_configuration/the_basics.html#include (Plus the .py) But, the install command needs to have the jupyter extra, so |
I'll do it in a separate PR, but I may not get to it before the next release. #dadlife Thank you for all your guidance. I like being able to contribute back. |
|
Also, again, if you're willing, mypy takes .py and .pyi: https://mypy.readthedocs.io/en/stable/command_line.html#specifying-what-to-type-check (see --exclude) Maybe you could do all in one PR too, if you're willing |
|
Pylint discovers .pyi files normally since 3.1 https://pylint.readthedocs.io/en/stable/whatsnew/3/3.1/index.html |
|
Isort can have more, including cython: https://pycqa.github.io/isort/docs/configuration/options.html#supported-extensions Config default: ['pxd', 'py', 'pyi', 'pyx'] |
Fixes #5979
Proposed Changes
The ruff tool supports *.py, *.pyi, and *.ipynb. This PR adds the last two file types to the ruff and ruff-format linters to allow these files to be checked in
list_of_filesmode.Readiness Checklist
Author/Contributor
Reviewing Maintainer
breakingif this is a large fundamental changeautomation,bug,documentation,enhancement,infrastructure, orperformance