Skip to content

Conversation

@srittau
Copy link
Contributor

@srittau srittau commented Jul 23, 2021

The test was accessing typing.{io,re}.all, which triggered the
warning. This check isn't necessary anymore, since the objects from
typing.{io,re}.all are in typing.all as well, since Python 3.10.

A NEWS should not be required, since this affects the tests only.

Cc @serhiy-storchaka

https://bugs.python.org/issue38291

The test was accessing typing.{io,re}.__all__, which triggered the
warning. This check isn't necessary anymore, since the objects from
typing.{io,re}.__all__ are in typing.__all__ as well, since Python 3.10.
@srittau srittau requested a review from gvanrossum as a code owner July 23, 2021 18:05
@bedevere-bot bedevere-bot added the tests Tests in the Lib/test dir label Jul 23, 2021
@srittau
Copy link
Contributor Author

srittau commented Jul 23, 2021

Aside: I could not reproduce the warning when running the tests locally.

@bedevere-bot
Copy link

@gvanrossum: Please replace # with GH- in the commit message next time. Thanks!

@srittau srittau deleted the fix-typing-warnings branch July 23, 2021 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants