Skip to content
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

bpo-41428: Documentation for PEP 604 #22517

Merged
merged 8 commits into from Oct 5, 2020

Conversation

Fidget-Spinner
Copy link
Member

@Fidget-Spinner Fidget-Spinner commented Oct 3, 2020

Added documentation for PEP 604.

https://bugs.python.org/issue41428

Doc/library/stdtypes.rst Outdated Show resolved Hide resolved
@gvanrossum gvanrossum removed the request for review from ilevkivskyi Oct 3, 2020
@gvanrossum
Copy link
Member

gvanrossum commented Oct 3, 2020

@MaggieMoss Do you have time to you review this? It's about your code. :-)

Doc/whatsnew/3.10.rst Outdated Show resolved Hide resolved
Doc/library/stdtypes.rst Outdated Show resolved Hide resolved
Doc/library/stdtypes.rst Outdated Show resolved Hide resolved
Doc/library/stdtypes.rst Outdated Show resolved Hide resolved
Doc/library/stdtypes.rst Outdated Show resolved Hide resolved
Doc/library/stdtypes.rst Outdated Show resolved Hide resolved
Doc/library/types.rst Outdated Show resolved Hide resolved
Doc/library/stdtypes.rst Outdated Show resolved Hide resolved
Doc/library/typing.rst Outdated Show resolved Hide resolved
Doc/whatsnew/3.10.rst Outdated Show resolved Hide resolved
@Fidget-Spinner
Copy link
Member Author

Fidget-Spinner commented Oct 4, 2020

@gvanrossum and @pablogsal , thanks for the extremely thorough review. I implemented the suggested changes.

Copy link
Member

@gvanrossum gvanrossum left a comment

These small updates, and the link to PEP 585 that you mentioned.

Doc/library/stdtypes.rst Outdated Show resolved Hide resolved
Doc/whatsnew/3.10.rst Show resolved Hide resolved
Doc/whatsnew/3.10.rst Outdated Show resolved Hide resolved
Copy link
Member

@gvanrossum gvanrossum left a comment

Yea!

@gvanrossum gvanrossum merged commit 8e1dd55 into python:master Oct 5, 2020
3 checks passed
@Fidget-Spinner
Copy link
Member Author

Fidget-Spinner commented Oct 5, 2020

Thanks guido, pablo and eamanu for the patience and guidance for this PR. This was my first time writing reST/writing docs for CPython for that matter, and it was extremely fun! The devguide was extremely helpful too :).

@gvanrossum
Copy link
Member

gvanrossum commented Oct 5, 2020

@Fidget-Spinner You did a great job. Do you want to help more?

@Fidget-Spinner
Copy link
Member Author

Fidget-Spinner commented Oct 5, 2020

@gvanrossum Yea definitely, I'll work on something else when I have the time to do so !

@Fidget-Spinner Fidget-Spinner deleted the add-pep604-docs branch Oct 5, 2020
xzy3 pushed a commit to xzy3/cpython that referenced this pull request Oct 18, 2020
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants