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

fix(nodebuilder): fix error log and shut the fx #2160

Merged
merged 1 commit into from
May 3, 2023

Conversation

Wondertan
Copy link
Member

@Wondertan Wondertan commented May 3, 2023

Granular change descriptions are in comments

nodebuilder/node.go Show resolved Hide resolved
nodebuilder/node.go Show resolved Hide resolved
logs/logs.go Show resolved Hide resolved
@codecov-commenter
Copy link

Codecov Report

Merging #2160 (2ddc9f8) into main (4d31ad4) will decrease coverage by 0.16%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main    #2160      +/-   ##
==========================================
- Coverage   56.12%   55.96%   -0.16%     
==========================================
  Files         213      213              
  Lines       13846    13847       +1     
==========================================
- Hits         7771     7750      -21     
- Misses       5303     5325      +22     
  Partials      772      772              
Impacted Files Coverage Δ
nodebuilder/node.go 57.35% <0.00%> (ø)
logs/logs.go 90.90% <100.00%> (+0.43%) ⬆️

... and 3 files with indirect coverage changes

@Wondertan Wondertan added the kind:fix Attached to bug-fixing PRs label May 3, 2023
@Wondertan Wondertan marked this pull request as ready for review May 3, 2023 16:51
@Wondertan Wondertan enabled auto-merge (squash) May 3, 2023 17:04
@Wondertan Wondertan merged commit e8ae799 into main May 3, 2023
16 of 19 checks passed
@Wondertan Wondertan deleted the hlib/node/fix-logging branch May 3, 2023 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:fix Attached to bug-fixing PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants