Skip to content

Update reference to the parser from LL(1) to PEG #767

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

Merged
merged 2 commits into from
Nov 14, 2021

Conversation

itamaro
Copy link
Contributor

@itamaro itamaro commented Nov 14, 2021

not sure if this is considered a minor fix, let me know if I should file an issue for this first

langchanges.rst Outdated
@@ -47,8 +47,8 @@ You will first need a PEP that you will present to python-ideas. You may be a
little hazy on the technical details as various core developers can help with
that, but do realize that if you do not present your idea to python-ideas or
python-list ahead of time you may find out it is technically not possible
(e.g., Python's parser will not support the grammar change as it is an LL(1)
parser). Expect extensive comments on the PEP, some of which will be negative.
(e.g., Python's parser will not support the grammar change as a PEG parser).
Copy link
Member

Choose a reason for hiding this comment

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

I think the whole sentence in parentheses should be removed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed and updated PR

@pablogsal pablogsal merged commit 3036184 into python:master Nov 14, 2021
@pablogsal
Copy link
Member

Thanks for the PR @itamaro !

AA-Turner pushed a commit to AA-Turner/devguide that referenced this pull request Jun 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants