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

Decouple nitro-contracts from nitro #47

Merged
merged 19 commits into from
Apr 24, 2024
Merged

Decouple nitro-contracts from nitro #47

merged 19 commits into from
Apr 24, 2024

Conversation

gvladika
Copy link
Contributor

@gvladika gvladika commented Mar 28, 2024

PR enables using custom nitro-contracts and token-bridge-contracts versions in testnode.

So far, nitro-contracts version depended on the pinned nitro-contracts commit in nitro. And Go deployer was used to deploy the chains. Now that's replaced with a script in nitro-contracts which will do equivalent - deploy RollupCreator, deploy rollup and store configs and addresses.

Versions are provided via NITRO_CONTRACTS_BRANCH and TOKEN_BRIDGE_BRANCH env vars

@gvladika gvladika marked this pull request as ready for review March 28, 2024 18:52
@gvladika gvladika requested review from gzeoneth and godzillaba March 29, 2024 09:21
rollupcreator/Dockerfile Outdated Show resolved Hide resolved
tokenbridge/Dockerfile Outdated Show resolved Hide resolved
@gvladika gvladika changed the base branch from master to release April 23, 2024 09:34
'e2e-to-ci' implements support for testnode to use custom nitro-contracts branches
Once 'e2e-to-ci' is in master, 'NITRO_CONTRACTS_BRANCH' env var can be removed from CI
@gvladika gvladika merged commit 9636355 into release Apr 24, 2024
1 check passed
Ferret-san pushed a commit to celestiaorg/nitro-testnode that referenced this pull request Aug 27, 2024
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.

3 participants