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

Shorten maintenance span #181

Merged
merged 2 commits into from
Apr 29, 2022
Merged

Shorten maintenance span #181

merged 2 commits into from
Apr 29, 2022

Conversation

vkgnosis
Copy link
Contributor

Looking at Kibana I realized the hash adds too much visual noise for
little benefit when it is prepended to every log message so I'm changing
this to just say block=1234 and adding back the previous log message
showing the hash.

Test Plan

observe changed output

Looking at Kibana I realized the hash adds too much visual noise for
little benefit when it is prepended to every log message so I'm changing
this to just say block=1234 and adding back the previous log message
showing the hash.
@vkgnosis vkgnosis requested a review from a team as a code owner April 29, 2022 08:57
@codecov-commenter
Copy link

codecov-commenter commented Apr 29, 2022

Codecov Report

Merging #181 (9f83e47) into main (a9eb00a) will increase coverage by 0.19%.
The diff coverage is 28.57%.

❗ Current head 9f83e47 differs from pull request most recent head 016759f. Consider uploading reports for the commit 016759f to get more accurate results

@@            Coverage Diff             @@
##             main     #181      +/-   ##
==========================================
+ Coverage   64.62%   64.82%   +0.19%     
==========================================
  Files         185      185              
  Lines       38567    38400     -167     
==========================================
- Hits        24924    24892      -32     
+ Misses      13643    13508     -135     

@vkgnosis vkgnosis enabled auto-merge (squash) April 29, 2022 09:20
@vkgnosis vkgnosis merged commit 255ad96 into main Apr 29, 2022
@vkgnosis vkgnosis deleted the tracing_ branch April 29, 2022 11:29
@github-actions github-actions bot locked and limited conversation to collaborators Apr 29, 2022
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.

3 participants