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(logs): bitswap logger names broke in latest version bump, updated #3713

Merged
merged 3 commits into from
Sep 10, 2024

Conversation

renaynay
Copy link
Member

@renaynay renaynay commented Sep 9, 2024

Latest version bump of boxo broke bitswap logger names, updated.

@renaynay renaynay added the kind:fix Attached to bug-fixing PRs label Sep 9, 2024
@renaynay renaynay self-assigned this Sep 9, 2024
cristaloleg
cristaloleg previously approved these changes Sep 9, 2024
@codecov-commenter
Copy link

codecov-commenter commented Sep 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.61%. Comparing base (2469e7a) to head (1137e84).
Report is 189 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3713      +/-   ##
==========================================
+ Coverage   44.83%   45.61%   +0.77%     
==========================================
  Files         265      281      +16     
  Lines       14620    16032    +1412     
==========================================
+ Hits         6555     7313     +758     
- Misses       7313     7877     +564     
- Partials      752      842      +90     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Wondertan
Wondertan previously approved these changes Sep 9, 2024
@renaynay renaynay dismissed stale reviews from Wondertan and cristaloleg via 8586572 September 9, 2024 14:44
@renaynay renaynay enabled auto-merge (squash) September 10, 2024 08:35
@renaynay renaynay merged commit d3f44e1 into celestiaorg:main Sep 10, 2024
25 of 27 checks passed
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.

5 participants