-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(node): remove test code from production code #876
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @matan-starkware and the rest of your teammates on Graphite |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #876 +/- ##
==========================================
- Coverage 74.29% 74.24% -0.05%
==========================================
Files 358 358
Lines 36280 36183 -97
Branches 36280 36183 -97
==========================================
- Hits 26953 26864 -89
+ Misses 7188 7182 -6
+ Partials 2139 2137 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
43edf05
to
2c9058c
Compare
2c9058c
to
980dfdb
Compare
50b0311
to
be41b48
Compare
980dfdb
to
6742696
Compare
be41b48
to
908c8f1
Compare
6742696
to
665bf3a
Compare
908c8f1
to
586d85b
Compare
665bf3a
to
2d3e7ed
Compare
586d85b
to
34931fe
Compare
2d3e7ed
to
69a29e7
Compare
34931fe
to
033822a
Compare
69a29e7
to
7d5e08b
Compare
033822a
to
15b50b5
Compare
7d5e08b
to
0dbb3fd
Compare
15b50b5
to
4385c24
Compare
0dbb3fd
to
c76dbad
Compare
4385c24
to
52be37b
Compare
c76dbad
to
d39e534
Compare
52be37b
to
8dd9c3c
Compare
a10b3df
to
e56d670
Compare
8dd9c3c
to
0e5ff51
Compare
e56d670
to
837da3d
Compare
0e5ff51
to
1ad069a
Compare
837da3d
to
333f99a
Compare
9af919c
to
050def1
Compare
Remove the test config and test setup from production code. This causes us to create a new run_consensus binary for the test flow.
050def1
to
470e815
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 3 of 7 files at r1, 1 of 1 files at r2, 5 of 5 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @matan-starkware)
Merge activity
|
This overrides the default by adding a network receiver to simulate a faulty network
This change is