Issue30143
Created on 2017-04-23 12:11 by serhiy.storchaka, last changed 2017-11-16 07:19 by serhiy.storchaka. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 1262 | merged | serhiy.storchaka, 2017-04-23 12:13 | |
| Messages (4) | |||
|---|---|---|---|
| msg292163 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * | Date: 2017-04-23 12:11 | |
Proposed patch makes 2to3 generating a code that uses abstract collection classes Sequence and Mapping from collections.abc rather than collections. Since abstract collection classes now are defined in collections.abc and collections contains only aliases for compatibility, this is more idiomatic Python 3 code. |
|||
| msg306279 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * | Date: 2017-11-15 16:03 | |
What are your thoughts about this Benjamin? |
|||
| msg306282 - (view) | Author: Raymond Hettinger (rhettinger) * | Date: 2017-11-15 16:33 | |
+1 I think this change will be essential. IIRC, elsewhere Guido approved a 3.7 change so that the ABC will no longer be visible in collections. |
|||
| msg306341 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * | Date: 2017-11-16 07:16 | |
New changeset 0a2abdfca2495291809855cf7dfc6721c9c962e3 by Serhiy Storchaka in branch 'master': bpo-30143: 2to3 now generates a code that uses abstract collection classes (#1262) https://github.com/python/cpython/commit/0a2abdfca2495291809855cf7dfc6721c9c962e3 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2017-11-16 07:19:42 | serhiy.storchaka | set | status: open -> closed resolution: fixed stage: patch review -> resolved |
| 2017-11-16 07:16:27 | serhiy.storchaka | set | messages: + msg306341 |
| 2017-11-15 16:33:29 | rhettinger | set | nosy:
+ rhettinger messages: + msg306282 |
| 2017-11-15 16:03:56 | serhiy.storchaka | set | messages: + msg306279 |
| 2017-04-23 12:13:57 | serhiy.storchaka | set | pull_requests: + pull_request1375 |
| 2017-04-23 12:11:25 | serhiy.storchaka | create | |