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

feat(client): simplify client #521

Merged
merged 11 commits into from
Jan 22, 2024
Merged

feat(client): simplify client #521

merged 11 commits into from
Jan 22, 2024

Conversation

mask-pp
Copy link
Contributor

@mask-pp mask-pp commented Jan 21, 2024

No description provided.

Copy link

codecov bot commented Jan 21, 2024

Codecov Report

Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (46779ca) 56.24% compared to head (a115600) 56.02%.

Files Patch % Lines
pkg/rpc/client.go 54.54% 5 Missing and 5 partials ⚠️
pkg/rpc/engine.go 50.00% 4 Missing and 2 partials ⚠️
pkg/rpc/ethclient.go 95.34% 1 Missing and 1 partial ⚠️
pkg/rpc/utils.go 50.00% 2 Missing ⚠️
pkg/rpc/methods.go 75.00% 1 Missing ⚠️
prover/proof_submitter/proof_contester.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #521      +/-   ##
==========================================
- Coverage   56.24%   56.02%   -0.22%     
==========================================
  Files          44       44              
  Lines        5576     5526      -50     
==========================================
- Hits         3136     3096      -40     
+ Misses       2117     2109       -8     
+ Partials      323      321       -2     

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

@davidtaikocha davidtaikocha merged commit bbe9ed7 into main Jan 22, 2024
4 checks passed
@davidtaikocha davidtaikocha deleted the fix/simplify_client branch January 22, 2024 08:50
davidtaikocha pushed a commit that referenced this pull request Jan 23, 2024
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