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

feat(driver): improve driver implementation #639

Merged
merged 8 commits into from
Mar 15, 2024
Merged

Conversation

davidtaikocha
Copy link
Member

No description provided.

Copy link

codecov bot commented Mar 15, 2024

Codecov Report

Attention: Patch coverage is 67.90123% with 26 lines in your changes are missing coverage. Please review.

Project coverage is 50.30%. Comparing base (090a466) to head (e39679a).

Files Patch % Lines
driver/chain_syncer/chain_syncer.go 40.00% 6 Missing ⚠️
pkg/rpc/methods.go 16.66% 5 Missing ⚠️
driver/chain_syncer/calldata/syncer.go 78.94% 3 Missing and 1 partial ⚠️
driver/chain_syncer/beaconsync/progress_tracker.go 25.00% 3 Missing ⚠️
driver/chain_syncer/beaconsync/syncer.go 0.00% 3 Missing ⚠️
driver/driver.go 75.00% 2 Missing and 1 partial ⚠️
prover/event_handler/transition_contested.go 0.00% 1 Missing ⚠️
prover/event_handler/transition_proved.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #639      +/-   ##
==========================================
+ Coverage   50.24%   50.30%   +0.06%     
==========================================
  Files          56       56              
  Lines        5784     5751      -33     
==========================================
- Hits         2906     2893      -13     
+ Misses       2565     2551      -14     
+ Partials      313      307       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@davidtaikocha davidtaikocha marked this pull request as ready for review March 15, 2024 09:26
@davidtaikocha davidtaikocha enabled auto-merge (squash) March 15, 2024 09:26
@davidtaikocha davidtaikocha merged commit fbd4d06 into main Mar 15, 2024
4 checks passed
@davidtaikocha davidtaikocha deleted the improve-driver branch March 15, 2024 09:42
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