Skip to content

ref(mcp): Inline _get_span_config() - #6761

Merged
alexander-alderman-webb merged 22 commits into
masterfrom
webb/mcp/use-middleware-2
Jul 28, 2026
Merged

ref(mcp): Inline _get_span_config()#6761
alexander-alderman-webb merged 22 commits into
masterfrom
webb/mcp/use-middleware-2

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Description

Remove unnecessary indirection by inlining _get_span_config().

Issues

Reminders

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

98741 passed | ⏭️ 6806 skipped | Total: 105547 | Pass Rate: 93.55% | Execution Time: 355m 22s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +52
Passed Tests 📈 +53
Failed Tests 📉 -1
Skipped Tests

➖ Removed Tests (1)

View removed tests
  • test_continuous_profiler_auto_start_and_stop_sampled[experiment-thread]
    • File: tests.profiler.test_continuous_profiler

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2559 uncovered lines.
✅ Project coverage is 89.56%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.56%    89.56%        —%
==========================================
  Files          193       193         —
  Lines        24531     24514       -17
  Branches      8678      8674        -4
==========================================
+ Hits         21971     21955       -16
- Misses        2560      2559        -1
- Partials      1400      1399        -1

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review July 7, 2026 11:17
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner July 7, 2026 11:17

@sentrivana sentrivana left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎖️

Base automatically changed from webb/mcp/use-middleware to master July 28, 2026 07:38
@alexander-alderman-webb
alexander-alderman-webb merged commit d55cfd3 into master Jul 28, 2026
145 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/mcp/use-middleware-2 branch July 28, 2026 07:52
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