-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
allowed-unused-imports
has incorrect documentation
#13668
Comments
We should allow defining this either at Typo fix at #13669 |
Thanks! Is there precedent for allowing other settings at two different levels? It seems a bit confusing. |
I'm not sure honestly. It does seem confusing if it's not necessary for some reason. cc @charliermarsh who reviewed that contribution. |
It should only be under lint.pyflakes. |
@tylerlaprade I'm unable to reproduce this. I created a |
@MichaReiser, hmm, these are the three options it gives me. I don't know how to distinguish between them. All three seem to have the same problem, though. |
What's the exact error message from the schema? Can you try commenting out all settings and then, one by one (or doing a binary search where you comment in 50%) comment in the sections? Do you see the same in a |
Oh! Good idea - it was hard to tell which line was problematic since all occurrences of |
The setting wasn't deprecated but I see you just solved it. Nice. Yeah, our error reporting for the lint section is unfortunate because we use some functionality to avoid repeating all options globally and under |
lint
section rather thanlint.pyflakes
The text was updated successfully, but these errors were encountered: