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

chore: add Taiko to the compatible chains list #4204

Merged
merged 3 commits into from
Feb 6, 2024

Conversation

eugenPtr
Copy link
Contributor

@eugenPtr eugenPtr commented Jan 30, 2024

Description

The verifier has been successfully tested on Taiko so I propose adding Taiko to the compatible chains list.

The test has been done following these steps

Here is the result of the test. The contracts and transactions can be reviewed on Taiko's explorer

Script ran successfully.

== Return ==
0: bool true

## Setting up 1 EVM.

==========================

Chain 167008

Estimated gas price: 3.000000002 gwei

Estimated total gas used for script: 3630073

Estimated amount required: 0.010890219007260146 ETH

==========================

###
Finding wallets for all the necessary addresses...
##
Sending transactions [0 - 1].
⠉ [00:00:00] [###########################################################################################################################################################################] 2/2 txes (0.0s)Transactions saved to: /mnt/d/noir-starter/with-foundry/broadcast/Verify.s.sol/167008/run-latest.json

Sensitive values saved to: /mnt/d/noir-starter/with-foundry/cache/Verify.s.sol/167008/run-latest.json

##
Waiting for receipts.
⠙ [00:00:19] [#######################################################################################################################################################################] 2/2 receipts (0.0s)
##### 167008
✅  [Success]Hash: 0x95002c05a742d36439add3765990e89d6e721da4320b745cd0256ed96abbcfee
Contract Address: 0xe9702d9635B5DEe8b95B9601eD0ecF4c2C2ddcaA
Block: 115470
Paid: 0.007670022002556674 ETH (2556674 gas * 3.000000001 gwei)


##### 167008
✅  [Success]Hash: 0xa69725345d121e58b46a397748a3195c5d25405ac940c72145e469105e8d567e
Contract Address: 0x3c90C07E9284d4322F713E684F673fD947fbf811
Block: 115470
Paid: 0.000709431000236477 ETH (236477 gas * 3.000000001 gwei)


Transactions saved to: /mnt/d/noir-starter/with-foundry/broadcast/Verify.s.sol/167008/run-latest.json

Sensitive values saved to: /mnt/d/noir-starter/with-foundry/cache/Verify.s.sol/167008/run-latest.json



==========================

ONCHAIN EXECUTION COMPLETE & SUCCESSFUL.
Total Paid: 0.008379453002793151 ETH (2793151 gas * avg 3.000000001 gwei)

Transactions saved to: /mnt/d/noir-starter/with-foundry/broadcast/Verify.s.sol/167008/run-latest.json

Sensitive values saved to: /mnt/d/noir-starter/with-foundry/cache/Verify.s.sol/167008/run-latest.json

@eugenPtr eugenPtr changed the title Add Taiko to the compatible chains list docs: add Taiko to the compatible chains list Jan 30, 2024
@eugenPtr eugenPtr changed the title docs: add Taiko to the compatible chains list chore: add Taiko to the compatible chains list Jan 30, 2024
@kevaundray kevaundray enabled auto-merge February 6, 2024 12:28
@kevaundray
Copy link
Contributor

Thanks for checking this for us!

@kevaundray kevaundray added this pull request to the merge queue Feb 6, 2024
Merged via the queue into noir-lang:master with commit c606447 Feb 6, 2024
29 of 31 checks passed
@vezenovm vezenovm mentioned this pull request Feb 6, 2024
5 tasks
github-merge-queue bot pushed a commit that referenced this pull request Feb 6, 2024
# Description

## Problem\*

Resolves <!-- Link to GitHub Issue -->

## Summary\*

master CI failing after #4204

## Additional Context



## Documentation\*

Check one:
- [ ] No documentation needed.
- [ ] Documentation included in this PR.
- [ ] **[Exceptional Case]** Documentation to be submitted in a separate
PR.

# PR Checklist\*

- [ ] I have tested the changes locally.
- [ ] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
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