Skip to content

bpo-44754: Spell out 's.pop() or s.pop(i)' in stdtypes.rst table#27398

Merged
ambv merged 1 commit intopython:mainfrom
sweeneyde:list_pop
Jul 28, 2021
Merged

bpo-44754: Spell out 's.pop() or s.pop(i)' in stdtypes.rst table#27398
ambv merged 1 commit intopython:mainfrom
sweeneyde:list_pop

Conversation

@sweeneyde
Copy link
Member

@sweeneyde sweeneyde commented Jul 28, 2021

Using [square brackets] to declare optional formal parameters in argument lists is okay in general, but the rest of this table shows prototypical usage, not formal declaration. There is already a clarifying footnote about the default case.

https://bugs.python.org/issue44754

@bedevere-bot bedevere-bot added the docs Documentation in the Doc dir label Jul 28, 2021
@sweeneyde sweeneyde changed the title bpo-44754: Spell out 's.pop() or s.pop(i)' bpo-44754: Spell out 's.pop() or s.pop(i)' in stdtypes.rst table Jul 28, 2021
@ambv ambv merged commit 2ff5bb4 into python:main Jul 28, 2021
@miss-islington
Copy link
Contributor

Thanks @sweeneyde for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.8, 3.9.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 28, 2021
(cherry picked from commit 2ff5bb4)

Co-authored-by: Dennis Sweeney <36520290+sweeneyde@users.noreply.github.com>
@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Jul 28, 2021
@bedevere-bot
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 28, 2021
(cherry picked from commit 2ff5bb4)

Co-authored-by: Dennis Sweeney <36520290+sweeneyde@users.noreply.github.com>
@bedevere-bot
Copy link

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

@bedevere-bot
Copy link

GH-27414 is a backport of this pull request to the 3.8 branch.

ambv pushed a commit that referenced this pull request Jul 28, 2021
(cherry picked from commit 2ff5bb4)

Co-authored-by: Dennis Sweeney <36520290+sweeneyde@users.noreply.github.com>
ambv pushed a commit that referenced this pull request Jul 28, 2021
(cherry picked from commit 2ff5bb4)

Co-authored-by: Dennis Sweeney <36520290+sweeneyde@users.noreply.github.com>
ambv pushed a commit that referenced this pull request Jul 28, 2021
(cherry picked from commit 2ff5bb4)

Co-authored-by: Dennis Sweeney <36520290+sweeneyde@users.noreply.github.com>
@sweeneyde sweeneyde deleted the list_pop branch August 25, 2021 01:21
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.

5 participants