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

[DO NOT MERGE] Symbols as weakmap keys - Stage 2 draft changes #1

Closed
wants to merge 8 commits into from

Conversation

acutmore
Copy link
Owner

@acutmore acutmore commented May 16, 2022

Moved to tc39#2777


Stage 2 specification text for Symbols as WeakMap keys for review.

The proposed spec has been written as a PR against the full spec due to the high number of small tweaks to non-normative descriptions to sections such as liveness.

A more minimal proposal specification with only the core API changes is available at http://tc39.es/proposal-symbols-as-weakmap-keys

Copy link

@mhofman mhofman left a comment

Choose a reason for hiding this comment

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

s/reference/referent/

A reference is the thing that points to the referent (the object or symbol)

spec.html Show resolved Hide resolved
spec.html Show resolved Hide resolved
spec.html Show resolved Hide resolved
spec.html Show resolved Hide resolved
spec.html Show resolved Hide resolved
@acutmore
Copy link
Owner Author

s/reference/referent/

A reference is the thing that points to the referent (the object or symbol)

Sounds good. Github is erroring whenever I try and commit the suggestions. "this diff has recently been updated"

image

I'll go old school and apply the suggestions manually

Co-authored-by: Mathieu Hofman <86499+mhofman@users.noreply.github.com>
@acutmore
Copy link
Owner Author

@leobalter has mentioned that it would be useful to get links to rendered previews of the spec. I agree.

I'm going to close this PR and re-open one but this time targeting the main upstream 262 repo. It looks like PRs there get previews generated automatically.

@acutmore acutmore closed this May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants