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-46113: Fixes in built in types documentation #30167

Merged
merged 5 commits into from Dec 18, 2021

Conversation

vivekvashist
Copy link
Contributor

@vivekvashist vivekvashist commented Dec 17, 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.

Thank you. Mostly looks good to me. For the sake of other reviewers, the changes are:

  • Make bullet point periods consistent
  • Make repl output more accurate. The current examples are slightly wrong.

I have a few questions below.

Doc/library/stdtypes.rst Show resolved Hide resolved
Doc/library/stdtypes.rst Show resolved Hide resolved
Copy link
Contributor

@asvetlov asvetlov left a comment

Choose a reason for hiding this comment

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

LGTM
@Fidget-Spinner I have no objections to the PR.
Please feel free to merge when you are ready.

@Fidget-Spinner Fidget-Spinner merged commit 6f2df42 into python:main Dec 18, 2021
@miss-islington
Copy link
Contributor

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 18, 2021
* FixGH-1 - isidentifier() function output

* FixGH-2 Update the str.splitlines() function parameter

* FixGH-3 Removed unwanted full stop for str and bytes types double quotes examples.

* FixGH-4 Updated class dict from **kwarg to **kwargs
(cherry picked from commit 6f2df42)

Co-authored-by: Vivek Vashist <vivekvashist@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 18, 2021
* FixGH-1 - isidentifier() function output

* FixGH-2 Update the str.splitlines() function parameter

* FixGH-3 Removed unwanted full stop for str and bytes types double quotes examples.

* FixGH-4 Updated class dict from **kwarg to **kwargs
(cherry picked from commit 6f2df42)

Co-authored-by: Vivek Vashist <vivekvashist@gmail.com>
@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Dec 18, 2021
@bedevere-bot
Copy link

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

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Dec 18, 2021
@bedevere-bot
Copy link

GH-30187 is a backport of this pull request to the 3.9 branch.

@Fidget-Spinner
Copy link
Member

Thanks for the review @asvetlov. It's always nice to have a second pair of eyes. Thank you @vivekvashist for fixing up the docs and answering my questions.

Fidget-Spinner pushed a commit that referenced this pull request Dec 18, 2021
* Fix-1 - isidentifier() function output

* Fix-2 Update the str.splitlines() function parameter

* Fix-3 Removed unwanted full stop for str and bytes types double quotes examples.

* Fix-4 Updated class dict from **kwarg to **kwargs
(cherry picked from commit 6f2df42)

Co-authored-by: Vivek Vashist <vivekvashist@gmail.com>
Fidget-Spinner pushed a commit that referenced this pull request Dec 18, 2021
* Fix-1 - isidentifier() function output

* Fix-2 Update the str.splitlines() function parameter

* Fix-3 Removed unwanted full stop for str and bytes types double quotes examples.

* Fix-4 Updated class dict from **kwarg to **kwargs
(cherry picked from commit 6f2df42)

Co-authored-by: Vivek Vashist <vivekvashist@gmail.com>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants