-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Check for pylint pragmas in __init__.py
#9496
Labels
Discussion 🤔
Enhancement ✨
Improvement to a component
Needs decision 🔒
Needs a decision before implemention or rejection
Comments
chaosflaws
added
the
Needs triage 📥
Just created, needs acknowledgment, triage, and proper labelling
label
Mar 11, 2024
Pierre-Sassoulas
added
Enhancement ✨
Improvement to a component
Discussion 🤔
Needs decision 🔒
Needs a decision before implemention or rejection
and removed
Needs triage 📥
Just created, needs acknowledgment, triage, and proper labelling
labels
Mar 12, 2024
We have a merge request open for per dir pylint configuration files #9395, which imo are better because it would be harder to miss that a package is affected by it and it would permit to configure more than just disable or enable. It would be easy to miss an |
Agreed, #9395 is the better solution, and is already in the works :-) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Discussion 🤔
Enhancement ✨
Improvement to a component
Needs decision 🔒
Needs a decision before implemention or rejection
Current problem
I would like to disable different pylint rules in my application code and my test code (specifically messages about certain missing docstrings).
Desired solution
There is an open issue for that (#2712), but I suppose this won't be supported soon. An easier solution to the problem I am facing would be to parse
__init__.py
files for pylint pragmas and, if present, apply them to all modules in that package.Additional context
IMHO, this should greatly reduce the need for #2712, since with this addition, all file- and block-related exclusions, file-based exclusions and directory-based exclusions would be supported.
The text was updated successfully, but these errors were encountered: