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

Remove superfluous ClassBody restriction #220

Merged
merged 1 commit into from Apr 10, 2024

Conversation

rbuckton
Copy link
Collaborator

@rbuckton rbuckton commented Mar 22, 2024

Per tc39/ecma262#3000 (review), the ClassBody restriction in the early errors for UsingDeclaration and AwaitUsingDeclaration is superfluous as it cannot occur, thus should be removed.

A PR against ecma262 is being tracked by rbuckton/ecma262#6.

Copy link

A preview of this PR can be found at https://tc39.es/proposal-explicit-resource-management/pr/220.

@rbuckton rbuckton added bug Something isn't working non-normative/editorial Indicates a non-normative/editorial change to the specification normative Indicates a normative change to the specification and removed non-normative/editorial Indicates a non-normative/editorial change to the specification labels Mar 22, 2024
@rbuckton
Copy link
Collaborator Author

This may be considered a normative change, but removing ClassBody from this list would be otherwise unobservable.

@rbuckton rbuckton added the needs-consensus A pull request that needs consensus at TC39 plenary label Mar 29, 2024
@rbuckton rbuckton added the has-consensus Indicates a pull request reached consensus at TC39 plenary. label Apr 10, 2024
@rbuckton rbuckton merged commit 6277c43 into main Apr 10, 2024
1 check passed
@rbuckton rbuckton deleted the remove-superfluous-classbody-restriction branch April 10, 2024 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working has-consensus Indicates a pull request reached consensus at TC39 plenary. needs-consensus A pull request that needs consensus at TC39 plenary normative Indicates a normative change to the specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants