Skip to content

settle inactive hermeses on node start - #5174

Merged
Guillembonet merged 1 commit into
masterfrom
settle-inactive
Jun 7, 2022
Merged

settle inactive hermeses on node start#5174
Guillembonet merged 1 commit into
masterfrom
settle-inactive

Conversation

@Guillembonet

Copy link
Copy Markdown
Contributor

Signed-off-by: Guillem Bonet guillem@mysterium.network

Signed-off-by: Guillem Bonet <guillem@mysterium.network>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #5174 (b7b1ae2) into master (14f70f7) will increase coverage by 0.01%.
The diff coverage is 62.50%.

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

@@            Coverage Diff             @@
##           master    #5174      +/-   ##
==========================================
+ Coverage   38.75%   38.76%   +0.01%     
==========================================
  Files         358      358              
  Lines       19356    19379      +23     
==========================================
+ Hits         7501     7512      +11     
- Misses      11120    11132      +12     
  Partials      735      735              
Impacted Files Coverage Δ
session/pingpong/hermes_url_getter.go 0.00% <ø> (ø)
session/pingpong/hermes_promise_settler.go 34.63% <62.50%> (+0.87%) ⬆️
requests/dialer_swarm.go 72.18% <0.00%> (-3.76%) ⬇️
p2p/channel.go 66.06% <0.00%> (-2.17%) ⬇️
session/pingpong/invoice_tracker.go 64.14% <0.00%> (-1.69%) ⬇️
nat/traversal/pinger.go 80.23% <0.00%> (+3.87%) ⬆️
core/policy/oracle.go 91.89% <0.00%> (+5.40%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 14f70f7...db60f6f. Read the comment docs.

@Guillembonet
Guillembonet merged commit 877a3ff into master Jun 7, 2022
@Guillembonet
Guillembonet deleted the settle-inactive branch June 7, 2022 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants