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

Normative: add #x in obj syntax #2290

Merged
merged 1 commit into from
Jul 14, 2021
Merged

Conversation

ljharb
Copy link
Member

@ljharb ljharb commented Jan 27, 2021

This PR is based on top of #1668. Please review by commit. As #1668 is updated and/or merged, I'll rebase this one.

@ljharb ljharb added normative change Affects behavior required to correctly evaluate some ECMAScript source text needs test262 tests The proposal should specify how to test an implementation. Ideally via github.com/tc39/test262 pending stage 4 This proposal has not yet achieved stage 4, but may otherwise be ready to merge. proposal This is related to a specific proposal, and will be closed/merged when the proposal reaches stage 4. labels Jan 27, 2021
@ljharb ljharb requested review from syg, michaelficarra, bakkot and a team January 27, 2021 22:19
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
spec.html Outdated Show resolved Hide resolved
@ljharb
Copy link
Member Author

ljharb commented Jan 28, 2021

I've split out all the fixes you requested that aren't part of this PR, but are instead part of #1668, into a separate commit. Please only review the last commit here :-)

@ljharb ljharb requested a review from jmdyck January 28, 2021 20:36
@jmdyck
Copy link
Collaborator

jmdyck commented Jan 28, 2021

Sorry, I wasn't paying enough attention (re #1668).

ljharb added a commit to ljharb/ecma262 that referenced this pull request Feb 5, 2021
spec.html Outdated Show resolved Hide resolved
ljharb added a commit to ljharb/ecma262 that referenced this pull request Feb 5, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request Feb 8, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request Mar 31, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request Apr 27, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request May 7, 2021
@WebReflection
Copy link

Thanks! Please ignore my comments then, I had a mix of a-aha and “the heck?” moment there. All good 👍

ljharb added a commit to ljharb/ecma262 that referenced this pull request Jun 29, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request Jun 29, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request Jun 29, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request Jun 29, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request Jun 29, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request Jun 29, 2021
@ljharb ljharb added has stage 4 This PR represents a proposal that has achieved stage 4, and is ready to merge. and removed pending stage 4 This proposal has not yet achieved stage 4, but may otherwise be ready to merge. labels Jul 14, 2021
ljharb added a commit to ljharb/ecma262 that referenced this pull request Jul 14, 2021
@ljharb ljharb requested a review from a team July 14, 2021 01:12
@bakkot bakkot added the ready to merge Editors believe this PR needs no further reviews, and is ready to land. label Jul 14, 2021
@ljharb ljharb merged commit 58e9d8d into tc39:master Jul 14, 2021
mathiasbynens pushed a commit to mathiasbynens/ecma262 that referenced this pull request Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has stage 4 This PR represents a proposal that has achieved stage 4, and is ready to merge. has test262 tests normative change Affects behavior required to correctly evaluate some ECMAScript source text proposal This is related to a specific proposal, and will be closed/merged when the proposal reaches stage 4. ready to merge Editors believe this PR needs no further reviews, and is ready to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants