Skip to content

bpo-41428: Documentation for PEP 604 #22517

Merged
gvanrossum merged 8 commits into
python:masterfrom
Fidget-Spinner:add-pep604-docs
Oct 5, 2020
Merged

bpo-41428: Documentation for PEP 604 #22517
gvanrossum merged 8 commits into
python:masterfrom
Fidget-Spinner:add-pep604-docs

Conversation

@Fidget-Spinner

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

Copy link
Copy Markdown
Member

Added documentation for PEP 604.

https://bugs.python.org/issue41428

Comment thread Doc/library/stdtypes.rst Outdated
@gvanrossum gvanrossum removed the request for review from ilevkivskyi October 3, 2020 15:44
@gvanrossum

Copy link
Copy Markdown
Member

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

Comment thread Doc/whatsnew/3.10.rst Outdated
Comment thread Doc/library/stdtypes.rst Outdated
Comment thread Doc/library/stdtypes.rst Outdated
Comment thread Doc/library/stdtypes.rst Outdated
Comment thread Doc/library/stdtypes.rst Outdated
Comment thread Doc/library/stdtypes.rst Outdated
Comment thread Doc/library/types.rst Outdated
Comment thread Doc/library/stdtypes.rst Outdated
Comment thread Doc/library/typing.rst Outdated
Comment thread Doc/whatsnew/3.10.rst Outdated
Comment thread Misc/NEWS.d/next/Documentation/2020-10-03-18-20-46.bpo-41428._ju1NE.rst Outdated
@Fidget-Spinner

Copy link
Copy Markdown
Member Author

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

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member

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.

Comment thread Doc/library/stdtypes.rst Outdated
Comment thread Doc/whatsnew/3.10.rst
Comment thread Doc/whatsnew/3.10.rst Outdated

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
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
Copy Markdown
Member

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

@Fidget-Spinner

Fidget-Spinner commented Oct 5, 2020

Copy link
Copy Markdown
Member Author

@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