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

fix(protocol): update ProofVerifier address name in AddressManager #13063

Merged
merged 2 commits into from
Jan 29, 2023

Conversation

davidtaikocha
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Jan 28, 2023

Codecov Report

Merging #13063 (c1c2147) into main (eb5d564) will decrease coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #13063      +/-   ##
==========================================
- Coverage   65.16%   64.97%   -0.20%     
==========================================
  Files         112      112              
  Lines        3095     3095              
  Branches      383      383              
==========================================
- Hits         2017     2011       -6     
- Misses       1001     1008       +7     
+ Partials       77       76       -1     
Flag Coverage Δ *Carryforward flag
bridge-ui 92.61% <ø> (ø) Carriedforward from 2227178
protocol 56.75% <ø> (-0.37%) ⬇️
relayer 69.10% <ø> (ø) Carriedforward from 2227178
ui 100.00% <ø> (ø) Carriedforward from 2227178

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
...es/protocol/contracts/thirdparty/LibMerkleTrie.sol 83.13% <0.00%> (-7.23%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@davidtaikocha davidtaikocha marked this pull request as ready for review January 28, 2023 06:13
@dantaik dantaik requested a review from dionysuzx January 29, 2023 02:27
@dantaik dantaik enabled auto-merge (squash) January 29, 2023 02:27
@dantaik dantaik requested review from dantaik and removed request for cyberhorsey January 29, 2023 03:23
@dantaik dantaik merged commit 4144f4b into main Jan 29, 2023
@dantaik dantaik deleted the update-deploy-l1-plonk-verifier branch January 29, 2023 03:24
@github-actions github-actions bot mentioned this pull request Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants