You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Replace --disallow-any flags with separate boolean flags. #4089
The --disallow-any flags interact in unexpected ways with other flags and they make configuration more complicated. We should replace them with a set of simple boolean flags. See discussion in #4076.
I think it's important that the documentation for the new disallow_any flags is grouped so it is easier to find other disallow_any flags if you're looking at one of them.
@JelleZijlstra I can try to implement this if you want -- it should be pretty straightforward. But if you want to do it -- feel free to.
Activity
gvanrossum commentedon Oct 13, 2017
When we do this, we should get rid of
--disallow-any=unannotatedcompletely, since it would just be a synonym for--disallow-untyped-defs.Fix config file parsing of disallow_any/disallow_untyped_defs combina…
JelleZijlstra commentedon Oct 13, 2017
#4076 already removes
unannotated.gvanrossum commentedon Oct 13, 2017
Are you sure? It still looks supported. Maybe you forgot to push a commit?
JelleZijlstra commentedon Oct 13, 2017
I think you're right. I'll submit another PR with the rest of my changes, probably not now though.
ilinum commentedon Oct 15, 2017
I think it's important that the documentation for the new
disallow_anyflags is grouped so it is easier to find otherdisallow_anyflags if you're looking at one of them.@JelleZijlstra I can try to implement this if you want -- it should be pretty straightforward. But if you want to do it -- feel free to.
Replace `--disallow-any` flags with separate boolean flags.
Replace `--disallow-any` flags with separate boolean flags. (#4178)