Skip to content

Conversation

@ilevkivskyi
Copy link
Member

@ilevkivskyi ilevkivskyi commented Aug 23, 2019

This PR enables --strict-equality and --show-error-codes for self-check, and adds error codes in some # type: ignores. So far this revealed two issues:

@msullivan
Copy link
Collaborator

I had a PR merged with pyflakes to fix the lint issues (PyCQA/pyflakes#455) but it hasn't been released yet I think

@ilevkivskyi
Copy link
Member Author

@msullivan This is great! We can then remove F821 from exception list when the new version is released.

@ilevkivskyi
Copy link
Member Author

I think this is quite uncontroversial, so I am going to merge this now.

@ilevkivskyi ilevkivskyi merged commit eb5f4a4 into python:master Aug 24, 2019
@ilevkivskyi ilevkivskyi deleted the tighten-self-check branch August 24, 2019 22:14
Michael0x2a pushed a commit to Michael0x2a/mypy that referenced this pull request Aug 25, 2019
This PR enables `--strict-equality` and `--show-error-codes` for self-check, and adds error codes in some `# type: ignore`s. So far this revealed two issues:
* Both PyCharm and flake8 get crazy about error codes, generating many false positives
* Error codes in `# type: ignore`s don't always work, see python#7384
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants