-
Notifications
You must be signed in to change notification settings - Fork 62
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
сhore: use upstream kurtosis package #57
Conversation
- name: Build Odyssey | ||
run: docker buildx build . --load -t ghcr.io/ithacaxyz/odyssey:latest | ||
- name: Run enclave | ||
id: kurtosis | ||
run: | | ||
git clone https://github.com/klkvr/optimism-package |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we still need to use your optimism-package fork?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
once ethpandaops/optimism-package#85 is merged there would be no need for it, can hold off this PR until it's merged
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah got it, no need to wait- we can just change it later was just curious
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Uses ethpandaops/optimism-package#85 for now, once merged can be replaced with upstream package.
This also switches to using images from https://github.com/paradigmxyz/optimism cc @Rjected