Skip to content

Comments

bitswap/client: propagate trace state when calling GetBlocks#1060

Merged
gammazero merged 2 commits intomainfrom
fix-trace-for-getblocks
Oct 28, 2025
Merged

bitswap/client: propagate trace state when calling GetBlocks#1060
gammazero merged 2 commits intomainfrom
fix-trace-for-getblocks

Conversation

@gammazero
Copy link
Contributor

@gammazero gammazero commented Oct 28, 2025

  • Additional trace fix for bitswap client GetBlocks (context)
  • Added comments to explain reason for code.

@codecov
Copy link

codecov bot commented Oct 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.10%. Comparing base (8c17f11) to head (a7d795a).
⚠️ Report is 3 commits behind head on main.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1060      +/-   ##
==========================================
+ Coverage   61.06%   61.10%   +0.04%     
==========================================
  Files         268      268              
  Lines       26132    26130       -2     
==========================================
+ Hits        15957    15967      +10     
+ Misses       8501     8491      -10     
+ Partials     1674     1672       -2     
Files with missing lines Coverage Δ
bitswap/client/client.go 78.65% <100.00%> (+2.07%) ⬆️

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gammazero gammazero changed the title Propagate trace state when calling GetBlocks bitswap/client: propagate trace state when calling GetBlocks Oct 28, 2025
@gammazero gammazero merged commit fc53cf8 into main Oct 28, 2025
17 checks passed
@gammazero gammazero deleted the fix-trace-for-getblocks branch October 28, 2025 16:06
@lidel lidel mentioned this pull request Oct 29, 2025
9 tasks
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.

2 participants