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

test: start orca contract with core-eval #4

Merged
merged 27 commits into from
Jun 26, 2024

Conversation

dckc
Copy link
Member

@dckc dckc commented Jun 25, 2024

I looked into helping with #3 . Reproducing those results seems to involve the whole multichain thing, but the error doesn't seem to have anything to do with interactions with other chains. I'd like to get this sort of thing working cleanly in ava tests, where we can use tools like a debugger.

For dependencies (not devDependencies), I started from scratch:

  • for @agoric/XYZ, I used dev versions: yarn add @agoric/XYZ@dev
  • for @endo/XYZ etc., I used yarn why in agoric-sdk to see what version to use
  • rather than import from @agoric/internal, I copied debug.js for makeTracer and copied the 1-line definition of StorageNodeShape
    • changed a bunch of console.log() to trace()
  • I narrowed the import for prepareRecorderKitMakers in order to avoid bundling all of @agoric/ertp

Each time I touched a file, format-on-save invoked prettier. Those edits are in separate style: commits.

DRAFT until:

  • check that deployment to an actual chain still works

When working on dapp-agoric-basics and dapp-offer-up, starting the chain takes just a few seconds. I'd like to know how to do that for the part of the multi-chain setup just for an agoric chain.

 - for @agoric/X, use @dev
 - for @endo/X etc., check `yarn why` in agoric-sdk
@dckc dckc requested a review from Jovonni June 25, 2024 18:38
dckc added 26 commits June 25, 2024 13:47
to avoid bringing in all of ERTP

bundle size before: 905582; after: 709624
(Sum of file sizes)
Jovonni added a commit that referenced this pull request Jun 25, 2024
@Jovonni Jovonni merged commit cf6e070 into dev/debugging-deployment Jun 26, 2024
0 of 2 checks passed
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