-
-
Notifications
You must be signed in to change notification settings - Fork 405
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
implement flake8-type-checking #2300
Conversation
df414de
to
9cc9c3e
Compare
Had to restrict it to python 3.8+ since flake8-type-checking doesn't support 3.7... |
Think I threw it on the todo pile in #1765 instead of just doing it immediately because there's no support for 3.7… But there's actually nothing that breaks if someone runs style checks on 3.7, only the potential for missing TC error codes that CI will flag. And probably no one should be running their main development on anything older than 3.8 or 3.9… so I declare my past self silly. Can you exclude/ignore the TC200 error series? We want to use delayed evaluation (it's why our files use (My present self is also silly, just for different reasons.) |
Cool. Time to ask for review from the one with actual experience using mypy. |
That's interesting. Maybe it's also time to officially add |
Could be, if it doesn't generate any errors after merging with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm OK with that as long as make mypy
works.
1731ae0
to
df815a2
Compare
Description
Tin. See #1765
Checklist
make qa
(runsmake quality
andmake test
)