-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Allow disabling bad-option-value
and unrecognized-option
with --disable=all
#6565
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
This will be quite tricky as this is dependent on the order in which things are parsed (and therefore likely impossible). Similar to my comment in #6556, we could try and create a system to defer I do think that we might want to update our documentation about this because I can understand that somebody will wonder about this one day. Edit: @jacobtylerwalls I might even consider this a |
Yeah this might turn out to be a won't fix and doc it, but I'll just leave it open long enough for someone (possibly me?) to spend a very short cycle making sure we're not missing anything obvious. |
https://docs.djangoproject.com/en/dev/releases/3.2.14/ Also removes native-string from the pylintrc_tweaks disable list, as it is no longer valid: pylint-dev/pylint#6565
https://docs.djangoproject.com/en/dev/releases/3.2.14/ Also removes native-string from the pylintrc_tweaks disable list, as it is no longer valid: pylint-dev/pylint#6565
https://docs.djangoproject.com/en/dev/releases/3.2.14/ Also removes native-string from the pylintrc_tweaks disable list, as it is no longer valid: pylint-dev/pylint#6565
Uh oh!
There was an error while loading. Please reload this page.
We should allow
--disable=all
to disable the messages emitted for apylintrc
, e.g.bad-option-value
andunrecognized-option
, as it's slightly unexpected to not be able to disable everything.The
pylintrc
used in the following run is found here.Originally posted by @jacobtylerwalls in #6556 (review)
The text was updated successfully, but these errors were encountered: