Skip to content

[3.10] bpo-46333: Honor module parameter in ForwardRef (GH-30536)#31379

Merged
miss-islington merged 1 commit into
python:3.10from
miss-islington:backport-6e7b813-3.10
Feb 17, 2022
Merged

[3.10] bpo-46333: Honor module parameter in ForwardRef (GH-30536)#31379
miss-islington merged 1 commit into
python:3.10from
miss-islington:backport-6e7b813-3.10

Conversation

@miss-islington

@miss-islington miss-islington commented Feb 17, 2022

Copy link
Copy Markdown
Contributor

The module parameter carries semantic information about the forward ref.
Forward refs are different if they refer to different module even if they
have the same name. This affects the __eq__, __repr__ and __hash__ methods.

Co-authored-by: Andreas Hangauer andreas.hangauer@siemens.com
Co-authored-by: Alex Waygood Alex.Waygood@Gmail.com
Co-authored-by: Ken Jin 28750310+Fidget-Spinner@users.noreply.github.com
(cherry picked from commit 6e7b813)

Co-authored-by: aha79 34090357+aha79@users.noreply.github.com

https://bugs.python.org/issue46333

Automerge-Triggered-By: GH:JelleZijlstra

The `module` parameter carries semantic information about the forward ref.
Forward refs are different if they refer to different module even if they
have the same name. This affects the `__eq__`, `__repr__` and `__hash__` methods.

Co-authored-by: Andreas Hangauer <andreas.hangauer@siemens.com>
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
Co-authored-by: Ken Jin <28750310+Fidget-Spinner@users.noreply.github.com>
(cherry picked from commit 6e7b813)

Co-authored-by: aha79 <34090357+aha79@users.noreply.github.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@aha79 and @JelleZijlstra: Status check is done, and it's a success ✅ .

@miss-islington

Copy link
Copy Markdown
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit a17d59a into python:3.10 Feb 17, 2022
@miss-islington miss-islington deleted the backport-6e7b813-3.10 branch February 17, 2022 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants