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

[spec] migrate global to Reflect methods #19

Merged
merged 1 commit into from
Sep 20, 2023
Merged

[spec] migrate global to Reflect methods #19

merged 1 commit into from
Sep 20, 2023

Conversation

ljharb
Copy link
Member

@ljharb ljharb commented Jun 2, 2023

No description provided.

@ljharb ljharb mentioned this pull request Jan 19, 2023
24 tasks
@bathos
Copy link
Contributor

bathos commented Jun 2, 2023

Oh wow — so %Reflect% should be understood going forward as a broad-purpose namespace for any kind of introspection & reflection, not just reflection of the object protocol?

(This seems unfortunate to me — there is value in %Reflect% outlining the shape of that protocol, appearing as a kind of default %Proxy% handler, and describing the precise minimum set of ops needed to reflect/explain every observable behavior of the language, which these are not. But I’m guessing this change came from a discussion that might have already weighed those things out and come to a different conclusion?)

@ljharb
Copy link
Member Author

ljharb commented Jun 2, 2023

Yes, that seemed to be the committee consensus when it was last discussed. https://github.com/tc39/notes/blob/7e73db5028979cb7afd85c0dd328e860015c074d/meetings/2023-01/feb-02.md?plain=1#L243-L248

@ljharb ljharb merged commit fa3d37e into main Sep 20, 2023
3 checks passed
@ljharb ljharb deleted the reflect branch September 20, 2023 17:31
@ljharb ljharb temporarily deployed to github-pages September 20, 2023 17:31 — with GitHub Pages Inactive
ljharb added a commit that referenced this pull request Sep 21, 2023
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.

None yet

2 participants