Skip to content

feat(pass-style): Safe promises can override @@toStringTag with a string#1785

Merged
erights merged 1 commit intomasterfrom
markm-label-promise-instances
Sep 26, 2023
Merged

feat(pass-style): Safe promises can override @@toStringTag with a string#1785
erights merged 1 commit intomasterfrom
markm-label-promise-instances

Conversation

@erights
Copy link
Copy Markdown
Contributor

@erights erights commented Sep 21, 2023

#1781 demonstrates that the current endo (and it just so happens, the already-released endo) enables a passable promise to carry the string that @FUDCo needs to support testing. However, its way of carrying a string was terribly obtuse.

This PR implements part of the plan explained at #1781 (comment) . Specifically, to have passable promises follow the precedent of passable far objects and exos, which also allow them to override the inherited @@toStringTag property with an own data property carrying a string value. However, agoric-sdk would not be able to use this more pleasant way of carrying a string until it is upgraded to depend on an endo containing this PR.

@erights erights requested review from FUDCo and mhofman September 21, 2023 02:24
@erights erights self-assigned this Sep 21, 2023
@erights erights force-pushed the markm-label-promise-instances branch 2 times, most recently from 0782780 to b4ddd1c Compare September 21, 2023 02:37
@erights erights force-pushed the markm-label-promise-instances branch from b4ddd1c to 55e094c Compare September 26, 2023 02:18
Copy link
Copy Markdown
Contributor

@FUDCo FUDCo left a comment

Choose a reason for hiding this comment

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

Yay!

@erights erights merged commit aebbea2 into master Sep 26, 2023
@erights erights deleted the markm-label-promise-instances branch September 26, 2023 03:04
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.

2 participants