Issue 33061: NoReturn missing from all in typing.py (original) (raw)
This issue has been migrated to GitHub: https://github.com/python/cpython/issues/77242
classification
Title: | NoReturn missing from __all__ in typing.py | ||
---|---|---|---|
Type: | behavior | Stage: | resolved |
Components: | Library (Lib) | Versions: | Python 3.8, Python 3.7, Python 3.6, Python 3.4, Python 3.5 |
process
Status: | closed | Resolution: | fixed |
---|---|---|---|
Dependencies: | Superseder: | ||
Assigned To: | Nosy List: | AllenTracht, gvanrossum, levkivskyi | |
Priority: | normal | Keywords: | patch |
Created on 2018-03-12 22:15 by AllenTracht, last changed 2022-04-11 14:58 by admin. This issue is now closed.
Pull Requests | |||
---|---|---|---|
URL | Status | Linked | Edit |
PR 6127 | merged | python-dev,2018-03-16 19:56 | |
PR 6162 | merged | levkivskyi,2018-03-20 22:17 |
Messages (6) | ||
---|---|---|
msg313962 - (view) | Author: Ivan Levkivskyi (levkivskyi) * ![]() |
Date: 2018-03-16 17:47 |
This is clearly a bug. Would you like to make a PR? | ||
msg313963 - (view) | Author: Guido van Rossum (gvanrossum) * ![]() |
Date: 2018-03-16 17:51 |
Yes please! Updating __all__ is too easy to forget. IIRC there are scripts that check for this but they are too strict. | ||
msg314111 - (view) | Author: Ivan Levkivskyi (levkivskyi) * ![]() |
Date: 2018-03-19 18:41 |
New changeset 4573820d2a9156346392838d455e89f33067e9dd by Ivan Levkivskyi (aetracht) in branch 'master': bpo-33061: Add missing 'NoReturn' to __all__ in typing.py (GH-6127) https://github.com/python/cpython/commit/4573820d2a9156346392838d455e89f33067e9dd | ||
msg314170 - (view) | Author: Ivan Levkivskyi (levkivskyi) * ![]() |
Date: 2018-03-20 22:07 |
I think it makes sense to also fix this in 3.7. I will prepare a backport now. | ||
msg314335 - (view) | Author: Ivan Levkivskyi (levkivskyi) * ![]() |
Date: 2018-03-23 21:44 |
New changeset ac5602746ed39ca6591e98e062e587121ac71371 by Ivan Levkivskyi in branch '3.7': bpo-33061: Add missing 'NoReturn' to __all__ in typing.py (GH-6127) (#6162) https://github.com/python/cpython/commit/ac5602746ed39ca6591e98e062e587121ac71371 | ||
msg314336 - (view) | Author: Ivan Levkivskyi (levkivskyi) * ![]() |
Date: 2018-03-23 21:47 |
The typing repo PR is not merged yet, but I am closing this here, the remainder is tracked in typing repo. |
History | |||
---|---|---|---|
Date | User | Action | Args |
2022-04-11 14:58:58 | admin | set | github: 77242 |
2018-03-23 21:47:40 | levkivskyi | set | status: open -> closedresolution: fixedmessages: + stage: patch review -> resolved |
2018-03-23 21:44:56 | levkivskyi | set | messages: + |
2018-03-20 22:17:35 | levkivskyi | set | pull_requests: + <pull%5Frequest5918> |
2018-03-20 22:07:52 | levkivskyi | set | messages: + |
2018-03-19 18:41:40 | levkivskyi | set | messages: + |
2018-03-16 19:56:27 | python-dev | set | keywords: + patchstage: patch reviewpull_requests: + <pull%5Frequest5889> |
2018-03-16 17:51:34 | gvanrossum | set | messages: + |
2018-03-16 17:47:54 | levkivskyi | set | nosy: + gvanrossum |
2018-03-16 17:47:38 | levkivskyi | set | nosy: + levkivskyimessages: + |
2018-03-12 22:15:08 | AllenTracht | create |