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

feat(driver): improve sync progress information #288

Merged
merged 4 commits into from
Jun 16, 2023

Conversation

davidtaikocha
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Jun 14, 2023

Codecov Report

Merging #288 (05c0561) into main (cdabcac) will decrease coverage by 1.01%.
The diff coverage is 20.45%.

❗ Current head 05c0561 differs from pull request most recent head 396fdf1. Consider uploading reports for the commit 396fdf1 to get more accurate results

@@            Coverage Diff             @@
##             main     #288      +/-   ##
==========================================
- Coverage   51.93%   50.93%   -1.01%     
==========================================
  Files          36       36              
  Lines        3843     3854      +11     
==========================================
- Hits         1996     1963      -33     
- Misses       1591     1634      +43     
- Partials      256      257       +1     
Impacted Files Coverage Δ
driver/chain_syncer/beaconsync/syncer.go 0.00% <0.00%> (ø)
driver/chain_syncer/beaconsync/progress_tracker.go 70.44% <33.33%> (-6.98%) ⬇️
driver/chain_syncer/chain_syncer.go 40.78% <41.66%> (-0.79%) ⬇️

... 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 marked this pull request as ready for review June 15, 2023 06:17
@davidtaikocha davidtaikocha merged commit 45d73b9 into main Jun 16, 2023
4 checks passed
@davidtaikocha davidtaikocha deleted the sync-progress-info branch June 16, 2023 00:44
@github-actions github-actions bot mentioned this pull request Jun 16, 2023
davidtaikocha added a commit that referenced this pull request Jun 16, 2023
Co-authored-by: Roger <50648015+RogerLamTd@users.noreply.github.com>
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.

fix: L2 execution engine is syncing SyncProgress
2 participants