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

refactor(driver/proposer): refactor some code in review #152

Closed
wants to merge 1 commit into from

Conversation

alexshliu
Copy link
Contributor

No description provided.

@alexshliu alexshliu force-pushed the refactor/some-review branch from 90a2a61 to 97e7587 Compare February 16, 2023 16:09
@github-actions
Copy link
Contributor

@taikoxyz taikoxyz deleted a comment from github-actions bot Feb 16, 2023
@codecov
Copy link

codecov bot commented Feb 16, 2023

Codecov Report

Merging #152 (97e7587) into main (f281121) will decrease coverage by 2.02%.
The diff coverage is 78.78%.

@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
- Coverage   58.32%   56.30%   -2.02%     
==========================================
  Files          34       35       +1     
  Lines        3323     3417      +94     
==========================================
- Hits         1938     1924      -14     
- Misses       1140     1253     +113     
+ Partials      245      240       -5     
Impacted Files Coverage Δ
bindings/encoding/input.go 40.18% <ø> (-0.56%) ⬇️
driver/chain_syncer/beaconsync/syncer.go 0.00% <0.00%> (ø)
driver/chain_syncer/calldata/syncer.go 49.37% <ø> (ø)
driver/driver.go 64.40% <50.00%> (+4.40%) ⬆️
driver/chain_syncer/chain_syncer.go 37.00% <71.42%> (+0.15%) ⬆️
driver/chain_syncer/beaconsync/progress_tracker.go 77.92% <82.85%> (ø)
driver/config.go 70.00% <100.00%> (ø)
proposer/pool_content_splitter.go 65.93% <100.00%> (ø)
proposer/proposer.go 59.29% <100.00%> (-0.36%) ⬇️
prover/prover.go 55.00% <0.00%> (-5.00%) ⬇️
... and 3 more

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

@alexshliu alexshliu changed the title refactor: refactor some code in review refactor(driver/proposer): refactor some code in review Feb 17, 2023
@alexshliu alexshliu marked this pull request as ready for review February 17, 2023 02:35
@alexshliu alexshliu self-assigned this Feb 17, 2023
@davidtaikocha
Copy link
Member

Changes are included in #153

@alexshliu alexshliu closed this Feb 17, 2023
@davidtaikocha davidtaikocha deleted the refactor/some-review branch February 17, 2023 09:53
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