-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
Replace "starred_list" with standard grammar term "expression_list" in for statement documentation #134026
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
Comments
Hi, I am willing to work on this issue #134026 |
Hi, I actually opened this issue because I noticed the inconsistency and proposed the solution. I'm planning to submit a PR for it shortly. Appreciate the interest, but I’ll be taking this one. I asked in the issues because even i am not sure if its supposed to be changed or whether "starred_list" is correct. Im hoping for someone more experienced to help clarify it. I am new here too and this is my first doc inconsistency, I do not have the power to assign issues as thats only for trusted developers i believe |
Got it, Thanks |
See Lines 715 to 723 in 6df3976
|
ah okay. Sorry to bother i must have missed it as i hit "starred" on my ctrl+f in the grammar. Thank you |
No, you misunderstood. This is a quote from the grammar for your reference. I believe issue is valid. |
oh okay.. its my first issue ever on github so am a bit nervous. |
I'm not sure about naming. But whatever it will be, you should define the starred_list. |
Oh okay. I will look into doing that |
Uh oh!
There was an error while loading. Please reload this page.
Documentation
In the compound statements docs, section 8.3 i found:
However, starred_list does not appear to be a defined production rule elsewhere in the language reference. It seems to act more like a semantic placeholder to indicate that starred_expressions (like *x) are allowed in the iterable.
Since the real grammar uses expression_list — which already includes starred_expression — would it be more consistent and accurate to write this instead?:
…and then note in the accompanying text that starred_expression is supported?
This might reduce confusion for readers trying to understand or implement the formal grammar.
Can I work on that?
Linked PRs
The text was updated successfully, but these errors were encountered: