Skip to content
This repository has been archived by the owner on Jul 5, 2023. It is now read-only.

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

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

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

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

@ilevkivskyi ilevkivskyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the backport!

Btw, when the next release of typed_ast is planned?

@gvanrossum
Copy link
Member

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

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@msullivan
Copy link
Collaborator Author

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
@msullivan msullivan deleted the ignores branch June 4, 2019 19:15
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 pushed 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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants