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

Editorial: Annex A tweaks #1868

Open
wants to merge 4 commits into
base: master
from
Open

Editorial: Annex A tweaks #1868

wants to merge 4 commits into from

Conversation

@jmdyck
Copy link
Collaborator

jmdyck commented Feb 9, 2020

This PR cherry-picks commits 4 - 7 of PR #1651. Their purpose is to reduce inconsistencies between the main body and Annex A (Grammar Summary), making it easier to keep them in synch going forward.

spec.html Show resolved Hide resolved
@jmdyck jmdyck force-pushed the jmdyck:annex_a_tweaks branch from 7e6748b to 698003b Feb 14, 2020
jmdyck added 4 commits Aug 11, 2019
... that I should have removed back in 573e382 (PR #370).
... for consistency.
(The vast majority of these are in Annex A.)
... to better match the order of productions in the main body.
We're talking about
    AsyncArrowFunction : CoverCallExpressionAndAsyncArrowHead
so change
    CoverParenthesizedExpressionAndArrowParameterList
to
    CoverCallExpressionAndAsyncArrowHead
@jmdyck jmdyck force-pushed the jmdyck:annex_a_tweaks branch from 698003b to 3dd9e0a Feb 20, 2020
@jmdyck

This comment has been minimized.

Copy link
Collaborator Author

jmdyck commented Feb 20, 2020

(force-pushed to resolve a merge conflict)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.