Thanks to visit codestin.com
Credit goes to github.com

Skip to content

bpo-45391: mark UnionType as a class in documentation #28757

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 9, 2021
Merged

bpo-45391: mark UnionType as a class in documentation #28757

merged 1 commit into from
Dec 9, 2021

Conversation

gaborbernat
Copy link
Contributor

@gaborbernat gaborbernat commented Oct 6, 2021

❯ sphobjinv suggest ./build/html/objects.inv UnionType
:py:class:`types.UnionType`

Signed-off-by: Bernát Gábor [email protected]

https://bugs.python.org/issue45391

Things to consider the type does show up as class now in documentation:
Screenshot 2021-10-06 at 13 43 14
vs before
Screenshot 2021-10-06 at 13 44 36

This PR only addresses the instance in the issue report, but we should discuss if we should change all such instances in this file (there are quite a few others suffering from the same problem).

@github-actions
Copy link

github-actions bot commented Nov 6, 2021

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Nov 6, 2021
@merwok merwok changed the title bpo-45391: UnionType is a class not data bpo-45391: make UnionType as a class in documentation Nov 6, 2021
@merwok merwok changed the title bpo-45391: make UnionType as a class in documentation bpo-45391: mark UnionType as a class in documentation Nov 6, 2021
Copy link
Member

@Fidget-Spinner Fidget-Spinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm merging this as this is more straightforward than the rest of the list mentioned on the bpo. Those need more deliberation.

@Fidget-Spinner Fidget-Spinner merged commit e2cfc89 into python:main Dec 9, 2021
@miss-islington
Copy link
Contributor

Thanks @gaborbernat for the PR, and @Fidget-Spinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 9, 2021
Signed-off-by: Bernát Gábor <[email protected]>
(cherry picked from commit e2cfc89)

Co-authored-by: Bernát Gábor <[email protected]>
@bedevere-bot
Copy link

GH-30004 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Dec 9, 2021
@Fidget-Spinner
Copy link
Member

Thanks @gaborbernat for catching this, and thank you @AlexWaygood for reminding me to check this PR.

miss-islington added a commit that referenced this pull request Dec 9, 2021
Signed-off-by: Bernát Gábor <[email protected]>
(cherry picked from commit e2cfc89)

Co-authored-by: Bernát Gábor <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news stale Stale PR or inactive for long period of time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants