-
Notifications
You must be signed in to change notification settings - Fork 12
devnet with rollups-contract:2.0.0-rc.16, forge and cannon #201
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
Conversation
🦋 Changeset detectedLatest commit: 0ed225f The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@types/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected] |
ef22f4f to
b330a91
Compare
b330a91 to
f1a66f1
Compare
|
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
temporarily remove ERC-4337 contracts
0242a14 to
0ed225f
Compare
This PR changes the devnet package to use forge and cannon, and define the import of cartesi-rollups-contracts through cannon publishing.
To not change so much I kept the same strategy of having a
@cartesi/devnetnpm package, just that its content format changed a bit. Here are the npm package files:note: the
anvil_state.jsonis a dump for chain 13370.We will need to then change things in the SDK and in the CLI, but we can first have the new devnet npm package published.
In the future we can reevaluate if we really need a devnet npm package. Maybe it's better to just have cannon projects of independent smart contracts (i.e.
test-tokens,alchemy-smart-account, etc, etc), and just use them in the build ofsdk.ps: this temporarily remove ERC-4337 contracts, need to handled in a future PR