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

fix: sync with shadows in agoric-sdk #1532

Merged
merged 1 commit into from
Apr 3, 2023
Merged

Conversation

erights
Copy link
Contributor

@erights erights commented Apr 1, 2023

Some code from the @agoric/store package was unbundled and migrated to the @endo/patterns and @endo/exo packages. However, until agoric-sdk is upgraded to depend on the endo with these migrated packages, it continues to use its own deprecated copies, which we be removed after this upgrade happens. To smooth the way for this, we try to keep the code in these packages in sync with their shadows in agoric-sdk.

However, since the last sync there has been version drift. This PR brings them back into alignment.

As verified by #1533 , this PR is independent of #1531 , and can proceed without interference.

@erights erights requested review from warner and michaelfig April 1, 2023 00:09
@erights erights self-assigned this Apr 1, 2023
@erights erights marked this pull request as ready for review April 1, 2023 00:23
Copy link
Contributor

@warner warner left a comment

Choose a reason for hiding this comment

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

I don't have strong opinions on this one.

@erights erights force-pushed the markm-resync-agoric-sdk branch from faffedc to b22b053 Compare April 1, 2023 02:38
@erights erights changed the base branch from master to markm-sync-with-exo-api-change April 1, 2023 02:39
Base automatically changed from markm-sync-with-exo-api-change to master April 1, 2023 02:51
@erights erights force-pushed the markm-resync-agoric-sdk branch from b22b053 to b9beeef Compare April 1, 2023 02:52
@erights erights requested a review from dckc April 3, 2023 21:37
Copy link
Contributor

@dckc dckc left a comment

Choose a reason for hiding this comment

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

I'm struggling to find the beginning of this story.

@turadg maybe you're more familiar?

@erights
Copy link
Contributor Author

erights commented Apr 3, 2023

I'm struggling to find the beginning of this story.

@turadg maybe you're more familiar?

#1532 is just repairing version drift that happened because the agoric-sdk shadow copies were updated without coordinated updates to the primary copies in endo. (edited)

packages/exo/src/exo-makers.js Outdated Show resolved Hide resolved
Copy link
Contributor

@dckc dckc left a comment

Choose a reason for hiding this comment

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

doesn't look like just syncing up, to me

packages/exo/src/exo-tools.js Show resolved Hide resolved
packages/patterns/src/types.js Show resolved Hide resolved
@erights erights force-pushed the markm-resync-agoric-sdk branch from 65e3a5e to 19e2833 Compare April 3, 2023 22:58
@erights erights enabled auto-merge April 3, 2023 22:59
@erights erights merged commit e9262de into master Apr 3, 2023
@erights erights deleted the markm-resync-agoric-sdk branch April 3, 2023 23:07
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