Skip to content

1366 Use ++ and ** operators in EBNF#1498

Merged
ndw merged 4 commits intoqt4cg:masterfrom
michaelhkay:1366-ebnf-for-separated-sequences
Oct 29, 2024
Merged

1366 Use ++ and ** operators in EBNF#1498
ndw merged 4 commits intoqt4cg:masterfrom
michaelhkay:1366-ebnf-for-separated-sequences

Conversation

@michaelhkay
Copy link
Contributor

Fix #1366

@michaelhkay michaelhkay added Editorial Minor typos, wording clarifications, example fixes, etc. Build Affects the build infrastructure only labels Oct 15, 2024
@michaelhkay michaelhkay requested a review from ndw as a code owner October 15, 2024 20:47
@michaelhkay
Copy link
Contributor Author

Fix #1348

Copy link
Contributor

@ndw ndw left a comment

Choose a reason for hiding this comment

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

Looks fine to me, and builds successfully.

@michaelhkay
Copy link
Contributor Author

Note that there's a high risk of conflict with any other PR that changes the grammar - will probably need to do some conflict resolution.

@ndw
Copy link
Contributor

ndw commented Oct 21, 2024

I was very tempted to merge this one early, so that PR authors could try to resolve that before tomorrow. But then I chickened out in case it introduced a lot of merge conflicts that authors wouldn't have time to fix.

@michaelhkay michaelhkay force-pushed the 1366-ebnf-for-separated-sequences branch from de6efaf to 57da5a0 Compare October 22, 2024 18:00
@ndw ndw merged commit a519271 into qt4cg:master Oct 29, 2024
@michaelhkay michaelhkay added the Completed PR has been applied, tests written and tagged, no further action needed label Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build Affects the build infrastructure only Completed PR has been applied, tests written and tagged, no further action needed Editorial Minor typos, wording clarifications, example fixes, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

In the EBNF, use explicit separator syntax

3 participants