Skip to content

minor changes to immutability constraints for Agent Record fields - #3700

Merged
ljharb merged 3 commits into
mainfrom
what-is-observed
Nov 4, 2025
Merged

ljharb merged 3 commits into
mainfrom
what-is-observed

Conversation

@michaelficarra

Copy link
Copy Markdown
Member

Three independent but related changes. I recommend reviewing as individual commits. I'm not sure about the status of the final commit, whether that was just a missed intention (and therefore can be handled by the editors) or would require consensus. Looking to get more insight from @syg at editor call.

@michaelficarra michaelficarra added the editor call to be discussed in the next editor call label Oct 3, 2025
@github-actions

github-actions Bot commented Oct 3, 2025

Copy link
Copy Markdown

The rendered spec for this PR is available at https://tc39.es/ecma262/pr/3700.

@bakkot bakkot 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 Oct 6, 2025
@bakkot

bakkot commented Oct 6, 2025 •

Copy link
Copy Markdown
Member

Editors believe the normative change was intended to be included in the BigInt PR, since it was in the proposal spec text (at the very bottom). As such, while it is a normative change, we believe it has consensus already and so does not need to come back to committee.

Also it's pretty much unimaginable that this could have been violated.

Comment thread spec.html Outdated
@michaelficarra

Copy link
Copy Markdown
Member Author

@ljharb Please add all 3 commits in this PR individually, do not squash.

@walkerheathr24-rgb

This comment was marked as spam.

@ljharb
ljharb merged commit f656b31 into main Nov 4, 2025
9 checks passed
@ljharb
ljharb deleted the what-is-observed branch November 4, 2025 05:12
@ljharb ljharb added normative change Affects behavior required to correctly evaluate some ECMAScript source text editorial change labels Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial change normative change Affects behavior required to correctly evaluate some ECMAScript source text ready to merge Editors believe this PR needs no further reviews, and is ready to land.

5 participants