Skip to content
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

Add arb_checkPublisherHealth method #898

Merged
merged 4 commits into from
Aug 4, 2022
Merged

Conversation

PlasmaPower
Copy link
Contributor

Also enables the arb namespace by default, and moves the revalidate block API to arbdebug.

@cla-bot cla-bot bot added the cla-signed label Aug 3, 2022
@PlasmaPower PlasmaPower added the node Changes the node software label Aug 3, 2022
@codecov
Copy link

codecov bot commented Aug 3, 2022

Codecov Report

Merging #898 (e3d01f5) into master (686b037) will decrease coverage by 4.30%.
The diff coverage is 33.33%.

❗ Current head e3d01f5 differs from pull request most recent head c3b2c0e. Consider uploading reports for the commit c3b2c0e to get more accurate results

@@            Coverage Diff             @@
##           master     #898      +/-   ##
==========================================
- Coverage   51.81%   47.50%   -4.31%     
==========================================
  Files         228      216      -12     
  Lines       28371    24282    -4089     
  Branches      502      502              
==========================================
- Hits        14701    11536    -3165     
+ Misses      12101    11169     -932     
- Partials     1569     1577       +8     

Copy link
Contributor

@hkalodner hkalodner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hkalodner hkalodner enabled auto-merge (rebase) August 4, 2022 19:17
@hkalodner hkalodner merged commit 2e26395 into master Aug 4, 2022
@hkalodner hkalodner deleted the arb-check-publisher-health branch August 5, 2022 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed node Changes the node software
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants