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

feat(driver): improve ResetL1Current method based on the latest protocol changes #445

Merged
merged 4 commits into from
Nov 3, 2023

Conversation

davidtaikocha
Copy link
Member

@davidtaikocha davidtaikocha commented Nov 3, 2023

Copy link

codecov bot commented Nov 3, 2023

Codecov Report

Merging #445 (54ebe86) into main (e688c25) will increase coverage by 0.19%.
The diff coverage is 81.25%.

@@            Coverage Diff             @@
##             main     #445      +/-   ##
==========================================
+ Coverage   61.22%   61.41%   +0.19%     
==========================================
  Files          44       44              
  Lines        5114     5101      -13     
==========================================
+ Hits         3131     3133       +2     
+ Misses       1673     1662      -11     
+ Partials      310      306       -4     
Files Coverage Δ
bindings/encoding/input.go 34.78% <100.00%> (ø)
bindings/encoding/struct.go 96.34% <ø> (ø)
proposer/proposer.go 61.83% <100.00%> (+0.42%) ⬆️
proposer/prover_selector/eth_fee_eoa_selector.go 69.87% <100.00%> (+0.19%) ⬆️
prover/prover.go 52.96% <100.00%> (ø)
prover/server/api.go 46.29% <100.00%> (+3.15%) ⬆️
driver/chain_syncer/chain_syncer.go 44.57% <0.00%> (ø)
driver/state/l1_current.go 65.71% <50.00%> (+4.69%) ⬆️

... and 1 file with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@davidtaikocha davidtaikocha marked this pull request as ready for review November 3, 2023 16:23
@davidtaikocha davidtaikocha enabled auto-merge (squash) November 3, 2023 16:24
@davidtaikocha davidtaikocha merged commit ddf6980 into main Nov 3, 2023
4 checks passed
@davidtaikocha davidtaikocha deleted the update-syncer branch November 3, 2023 19:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants