Skip to content
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

Support text appearing after "# type: ignore" #116

Merged
merged 2 commits into from Jun 4, 2019
Merged

Support text appearing after "# type: ignore" #116

merged 2 commits into from Jun 4, 2019

Conversation

@msullivan
Copy link
Collaborator

@msullivan msullivan commented May 22, 2019

This is to allow things like # type: ignore[E1000].
Essentially a backport of GH-13238, GH-13479, and GH-13504 from cpython.

@msullivan
Copy link
Collaborator Author

@msullivan msullivan commented May 22, 2019

Failure is because we still do 3.4 windows builds so I'm gonna put up a PR to turn those off and then rebase this

msullivan added 2 commits May 22, 2019
This is to allow things like `# type: ignore[E1000]`.
Essentially a backport of GH-13238, GH-13479, and GH-13504 from cpython.
@msullivan msullivan force-pushed the ignores branch from 264a93a to bfcef5e May 22, 2019
Copy link
Collaborator

@ilevkivskyi ilevkivskyi left a comment

Thanks for the backport!

Btw, when the next release of typed_ast is planned?

@gvanrossum
Copy link
Member

@gvanrossum gvanrossum commented Jun 2, 2019

Whoops, I missed this somehow. It'll take some time to review carefully. Maybe we can do another typed_ast release after this has landed? The release process is pretty straightforward and there are working instructions in in release_process.md.

Copy link
Member

@gvanrossum gvanrossum left a comment

All LGTM. Have you at least manually checked ast27? IIRC the tests only run for ast3.

@msullivan
Copy link
Collaborator Author

@msullivan msullivan commented Jun 4, 2019

The tests live in the ast3 directory but do actually contain some ast27 tests (including ones I wrote for this)

@msullivan msullivan merged commit 6222c24 into master Jun 4, 2019
4 checks passed
4 checks passed
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@msullivan msullivan deleted the ignores branch Jun 4, 2019
msullivan added a commit to msullivan/pyflakes that referenced this pull request Jul 19, 2019
This is to support allowing typecheckers to implement ignores for
specific errors, using syntax like `# type: ignore=E1000` or
`# type: ignore[type-mismatch` or some such.
mypy is about to add support for ignoring specific errors following
this design: python/mypy#7239

Support for extra text in type comments was implemented
in CPython as https://bugs.python.org/issue36878
and in typed_ast as python/typed_ast#116.
msullivan added a commit to msullivan/pyflakes that referenced this pull request Jul 19, 2019
This is to support allowing typecheckers to implement ignores for
specific errors, using syntax like `# type: ignore=E1000` or
`# type: ignore[type-mismatch` or some such.
mypy is about to add support for ignoring specific errors following
this design: python/mypy#7239

Support for extra text in type comments was implemented
in CPython as https://bugs.python.org/issue36878
and in typed_ast as python/typed_ast#116.
msullivan added a commit to msullivan/pyflakes that referenced this pull request Jul 19, 2019
This is to support allowing typecheckers to implement ignores for
specific errors, using syntax like `# type: ignore=E1000` or
`# type: ignore[type-mismatch` or some such.
mypy is about to add support for ignoring specific errors following
this design: python/mypy#7239

Support for extra text in type comments was implemented
in CPython as https://bugs.python.org/issue36878
and in typed_ast as python/typed_ast#116.
sigmavirus24 added a commit to PyCQA/pyflakes that referenced this pull request Aug 12, 2019
* In PEP 484 type comments, allow text after "# type: ignore"

This is to support allowing typecheckers to implement ignores for
specific errors, using syntax like `# type: ignore=E1000` or
`# type: ignore[type-mismatch` or some such.
mypy is about to add support for ignoring specific errors following
this design: python/mypy#7239

Support for extra text in type comments was implemented
in CPython as https://bugs.python.org/issue36878
and in typed_ast as python/typed_ast#116.

* add test back
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.