Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

fix(prover): Submit L2 signal service root instead of L1 when submitting proof #219

Merged
merged 9 commits into from
May 12, 2023

Conversation

cyberhorsey
Copy link
Contributor

No description provided.

@cyberhorsey cyberhorsey changed the title fix(prover): Submit L1 signal service root instead of L2 when submitting proof fix(prover): Submit L2 signal service root instead of L1 when submitting proof May 11, 2023
@codecov
Copy link

codecov bot commented May 11, 2023

Codecov Report

Merging #219 (f1cbab4) into main (21d7e78) will decrease coverage by 0.46%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##             main     #219      +/-   ##
==========================================
- Coverage   53.07%   52.61%   -0.46%     
==========================================
  Files          36       36              
  Lines        3416     3419       +3     
==========================================
- Hits         1813     1799      -14     
- Misses       1367     1384      +17     
  Partials      236      236              
Impacted Files Coverage Δ
pkg/rpc/methods.go 35.29% <0.00%> (ø)
prover/proof_producer/special_proof_producer.go 7.76% <0.00%> (-0.24%) ⬇️
prover/proof_submitter/valid_proof_submitter.go 61.50% <50.00%> (ø)
...ver/anchor_tx_constructor/anchor_tx_constructor.go 53.52% <100.00%> (ø)

... and 1 file with indirect coverage changes

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

@davidtaikocha davidtaikocha merged commit 74fe156 into main May 12, 2023
@davidtaikocha davidtaikocha deleted the signal_root_fix branch May 12, 2023 01:43
vhjiang pushed a commit to taikoverse/taiko-degen-client that referenced this pull request Jul 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants