Repository navigation
Simplify implementation of the union type #88894
Description
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = None
closed_at = <Date 2021-07-24.14:50:23.547>
created_at = <Date 2021-07-24.07:04:29.579>
labels = ['interpreter-core', 'type-feature', '3.11']
title = 'Simplify implementation of the union type'
updated_at = <Date 2021-07-24.14:50:23.546>
user = 'https://github.com/serhiy-storchaka'bugs.python.org fields:
activity = <Date 2021-07-24.14:50:23.546>
actor = 'pablogsal'
assignee = 'none'
closed = True
closed_date = <Date 2021-07-24.14:50:23.547>
closer = 'pablogsal'
components = ['Interpreter Core']
creation = <Date 2021-07-24.07:04:29.579>
creator = 'serhiy.storchaka'
dependencies = []
files = []
hgrepos = []
issue_num = 44731
keywords = ['patch']
message_count = 4.0
messages = ['398119', '398124', '398155', '398160']
nosy_count = 5.0
nosy_names = ['gvanrossum', 'serhiy.storchaka', 'pablogsal', 'uriyyo', 'kj']
pr_nums = ['27318', '27334']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue44731'
versions = ['Python 3.11']
Activity
serhiy-storchaka commentedon Jul 24, 2021
The proposed PR simplifies implementation of the union type by removing direct support of typing types. It was not necessary because all these types implement __or__ and __ror__ methods.
The only visible difference is that int | TypeVar() etc returns now typing.Union instead of types.Union. But TypeVar() | int already returns typing.Union.
serhiy-storchaka commentedon Jul 24, 2021
It fixes also bpo-44642.
pablogsal commentedon Jul 24, 2021
New changeset 0828423 by Serhiy Storchaka in branch 'main':
bpo-44731: Simplify the union type implementation (GH-27318)
0828423
pablogsal commentedon Jul 24, 2021
New changeset ca5a4cf by Ken Jin in branch '3.10':
bpo-44731: Simplify the union type implementation (GH-27318) (GH-27334)
ca5a4cf