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

Clarify ToIndex, ToLength, and ToIntegerOrInfinity #2501

Merged
merged 3 commits into from
Sep 22, 2021

Conversation

gibson042
Copy link
Contributor

No description provided.

spec.html Outdated Show resolved Hide resolved
@gibson042 gibson042 force-pushed the 2021-08-refactor-index-conversion branch from 89dca12 to f128a82 Compare August 27, 2021 16:38
spec.html Outdated Show resolved Hide resolved
@bakkot bakkot added the editor call to be discussed in the next editor call label Sep 8, 2021
Copy link
Contributor

@bakkot bakkot left a comment

Choose a reason for hiding this comment

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

This LGTM with my suggested change applied.

@michaelficarra michaelficarra added ready to merge Editors believe this PR needs no further reviews, and is ready to land. and removed editor call to be discussed in the next editor call labels Sep 22, 2021
@ljharb ljharb force-pushed the 2021-08-refactor-index-conversion branch from ee0b775 to 1822f37 Compare September 22, 2021 21:40
@ljharb ljharb merged commit 1822f37 into tc39:master Sep 22, 2021
mathiasbynens pushed a commit to mathiasbynens/ecma262 that referenced this pull request Oct 18, 2021
mathiasbynens pushed a commit to mathiasbynens/ecma262 that referenced this pull request Oct 18, 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
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.

4 participants