-
-
Notifications
You must be signed in to change notification settings - Fork 377
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
[docs] say that both i-control-code
and i-dont-control-code
are both True
by default
#3059
Comments
Thanks! @nifadyev, are you interested in working on this? |
By default we assume that people control their code :) |
But the explanation refers to the calling code, not the one being linted, eh? |
Right, this also needs to be clarified. Thanks! |
The idea is that you use |
Honestly, in the spirit of the implemented readability rules, that setting could've been called after what it does instead of a situation it's supposed to be set in. That would make it obvious why one would want to use it. |
Hey @sobolevn , is it right that setting should be renamed to be more explicit? Should one of these settings be set to |
@nifadyev I agree that every rule should have its own setting. But, on the other hand, I don't want to affect users that already use this rule with the old setting. So, the plan is:
|
Thanks for the plan, i will try to work on it in next weeks |
What's wrong
$sbj is confusing. Which is it?
How it should be
https://wemake-python-styleguide.rtfd.io/en/latest/pages/usage/configuration.html should not claim two opposite things being true.
Flake8 version and plugins
N/A
pip information
N/A
OS information
N/A
The text was updated successfully, but these errors were encountered: