Skip to content

gh-138970: Add general metadata system to the peg generator#138971

Merged
pablogsal merged 3 commits into
python:mainfrom
pablogsal:gh-138970
Sep 18, 2025
Merged

gh-138970: Add general metadata system to the peg generator#138971
pablogsal merged 3 commits into
python:mainfrom
pablogsal:gh-138970

Conversation

@pablogsal

@pablogsal pablogsal commented Sep 16, 2025

Copy link
Copy Markdown
Member

@pablogsal

Copy link
Copy Markdown
Member Author

I will use this later to make the invalid rules non implicit

@pablogsal pablogsal force-pushed the gh-138970 branch 2 times, most recently from f471328 to f55196f Compare September 16, 2025 10:30

@lysnikolaou lysnikolaou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me! That's gonna be useful! Left some minor comments.

Comment thread Tools/peg_generator/pegen/metagrammar.gram Outdated
Comment thread Lib/test/test_peg_generator/test_pegen.py Outdated
Comment thread Lib/test/test_peg_generator/test_pegen.py Outdated

@lysnikolaou lysnikolaou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Minor comments.

Comment thread Lib/test/test_peg_generator/test_pegen.py Outdated
Comment thread Lib/test/test_peg_generator/test_pegen.py Outdated

@lysnikolaou lysnikolaou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Thanks!

@pablogsal pablogsal merged commit 0ce9fb7 into python:main Sep 18, 2025
55 checks passed
@pablogsal pablogsal deleted the gh-138970 branch September 18, 2025 01:17
@encukou

encukou commented Sep 18, 2025

Copy link
Copy Markdown
Member

Turns out we only run the tests on Buildbots.
Fix in #139107

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