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 October 3, 2020 15:44
@gvanrossum
Copy link
Member

@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

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

Copy link
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

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

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

Choose a reason for hiding this comment

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

Yea!

@Fidget-Spinner
Copy link
Member Author

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

@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 October 5, 2020 15:40
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.

6 participants