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

gh-122845: fix parameter_list_starargs in function definition pseudo-grammar #122847

Merged
merged 3 commits into from
Jan 21, 2025

Conversation

skirpichev
Copy link
Member

@skirpichev skirpichev commented Aug 9, 2024

Thanks to Artur Chakhvadze for bugfix.


📚 Documentation preview 📚: https://cpython-previews--122847.org.readthedocs.build/

@bedevere-app bedevere-app bot added docs Documentation in the Doc dir skip news awaiting review labels Aug 9, 2024
@skirpichev skirpichev changed the title Fix parameter_list_starargs in function definition pseudo-grammar gh-122845: fix parameter_list_starargs in function definition pseudo-grammar Aug 9, 2024
@skirpichev
Copy link
Member Author

CC @terryjreedy, involved in d.p.o thread.

@terryjreedy
Copy link
Member

My involvement was that I misread the existing grammar ;-), and agreed that there seemed to be a problem, but I will not judge the proposed replacement. There are better people to do so.

@skirpichev skirpichev added needs backport to 3.12 bug and security fixes needs backport to 3.13 bugs and security fixes labels Oct 9, 2024
@skirpichev
Copy link
Member Author

Maybe @pablogsal ?

@pablogsal
Copy link
Member

This is correct as allows the missing

def foo(*, a, b, **kwargs):  # Valid - forces keyword-only arguments
    pass

and disallows bare *

@pablogsal pablogsal merged commit 6105846 into python:main Jan 21, 2025
29 checks passed
@miss-islington-app
Copy link

Thanks @skirpichev for the PR, and @pablogsal for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 21, 2025
…seudo-grammar (pythonGH-122847)

Thanks to Artur Chakhvadze for bugfix.
(cherry picked from commit 6105846)

Co-authored-by: Sergey B Kirpichev <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 21, 2025
…seudo-grammar (pythonGH-122847)

Thanks to Artur Chakhvadze for bugfix.
(cherry picked from commit 6105846)

Co-authored-by: Sergey B Kirpichev <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Jan 21, 2025

GH-129150 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jan 21, 2025
@bedevere-app
Copy link

bedevere-app bot commented Jan 21, 2025

GH-129151 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label Jan 21, 2025
pablogsal pushed a commit that referenced this pull request Jan 21, 2025
…pseudo-grammar (GH-122847) (#129151)

gh-122845: fix parameter_list_starargs in function definition pseudo-grammar (GH-122847)

Thanks to Artur Chakhvadze for bugfix.
(cherry picked from commit 6105846)

Co-authored-by: Sergey B Kirpichev <[email protected]>
pablogsal pushed a commit that referenced this pull request Jan 21, 2025
…pseudo-grammar (GH-122847) (#129150)

gh-122845: fix parameter_list_starargs in function definition pseudo-grammar (GH-122847)

Thanks to Artur Chakhvadze for bugfix.
(cherry picked from commit 6105846)

Co-authored-by: Sergey B Kirpichev <[email protected]>
@skirpichev skirpichev deleted the fundocs-122845 branch January 22, 2025 00:46
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
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants