Skip to content

ref(mcp): Inline _get_span_config()#6761

Open
alexander-alderman-webb wants to merge 5 commits into
webb/mcp/use-middlewarefrom
webb/mcp/use-middleware-2
Open

ref(mcp): Inline _get_span_config()#6761
alexander-alderman-webb wants to merge 5 commits into
webb/mcp/use-middlewarefrom
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 📊

90034 passed | ⏭️ 6302 skipped | Total: 96336 | Pass Rate: 93.46% | Execution Time: 319m 14s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -11
Passed Tests 📉 -11
Failed Tests
Skipped Tests

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2480 uncovered lines.
❌ Project coverage is 89.62%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    89.73%    89.62%    -0.11%
==========================================
  Files          192       192         —
  Lines        23823     23898       +75
  Branches      8226      8242       +16
==========================================
+ Hits         21378     21418       +40
- Misses        2445      2480       +35
- Partials      1349      1352        +3

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.

🎖️

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