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

Mitigate more ag-solo robustness issues #3808

Merged
merged 5 commits into from
Sep 10, 2021
Merged

Conversation

michaelfig
Copy link
Member

The three commits are mostly independent and can be read separately.

@michaelfig michaelfig added captp package: captp agoric-cli package: agoric-cli wallet labels Sep 9, 2021
@michaelfig michaelfig requested a review from dckc September 9, 2021 18:07
@michaelfig michaelfig self-assigned this Sep 9, 2021
Copy link
Member

@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.

grant access to the clock explicitly, please

packages/agoric-cli/lib/deploy.js Outdated Show resolved Hide resolved
packages/solo/src/scratch.js Show resolved Hide resolved
Copy link
Member

@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.

good stuff.

@michaelfig michaelfig merged commit 4d1007c into master Sep 10, 2021
@michaelfig michaelfig deleted the mfig-solo-robustness2 branch September 10, 2021 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agoric-cli package: agoric-cli captp package: captp wallet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CapTP Cannot read property 'res' of undefined
2 participants