Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

removed some double log lines #5067

Merged
merged 1 commit into from
Jan 22, 2020
Merged

removed some double log lines #5067

merged 1 commit into from
Jan 22, 2020

Conversation

maaktweluit
Copy link
Contributor

While debugging the b0.22 branch i found these log messages are duplicated and can be removed to make the logs cleaner

  • Task started was printed by rpc and task-manager
  • Task header was printed before and after the warning(info) message

Copy link
Contributor

@shadeofblue shadeofblue left a comment

Choose a reason for hiding this comment

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

guess it's okay after all

@codecov
Copy link

codecov bot commented Jan 22, 2020

Codecov Report

Merging #5067 into b0.22 will increase coverage by 0.19%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##            b0.22    #5067      +/-   ##
==========================================
+ Coverage   89.83%   90.03%   +0.19%     
==========================================
  Files         234      234              
  Lines       21999    21996       -3     
==========================================
+ Hits        19762    19803      +41     
+ Misses       2237     2193      -44

@shadeofblue shadeofblue merged commit 6427947 into b0.22 Jan 22, 2020
@maaktweluit maaktweluit deleted the mwu/double-logs branch February 11, 2020 13:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants