-
Notifications
You must be signed in to change notification settings - Fork 34
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
op-server: bump celestia-openrpc to v0.3.0 #232
Conversation
46af593
to
f74bebb
Compare
f74bebb
to
3628f31
Compare
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, what's the main reason to move to mocha over staying with arabica? Wouldn't it make sense to be on arabica-10?
I wanted to test for a partner who is running against |
hmm, should we use mocha as default as it is the closest to a "production" environment? |
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.
utACK - I will need to sync (async is okay) with you to update the docs for this after merged and released
does this also close these issues @tuxcanfly ? |
I think it would be good to bump this to the latest version of celestia-node & local-celestia-devnet v0.12.0 |
* fix: testnet-up * Update docker-compose-testnet.yml
alternatively, maybe it would make more sense to ship this and then upgrade in a subsequent pr. not sure what is best |
Going ahead with merging this as it's blocking integration with |
Overview
This PR upgrades
celestia-openrpc
tov0.3.0
so that it is compatible withcelestia-node
v0.11.0
.While here, also update the testnet script for
mocha-4
.Resolves #233
Checklist