Skip to content

chore: Drop eventlet from conftest - #7060

Merged
sentrivana merged 1 commit into
major/3.0from
ivana/major/remove-eventlet
Aug 6, 2026
Merged

chore: Drop eventlet from conftest#7060
sentrivana merged 1 commit into
major/3.0from
ivana/major/remove-eventlet

Conversation

@sentrivana

Copy link
Copy Markdown
Contributor

Description

The actual compat code was removed with the contextvars shim. Just test utils remain.

Issues

Closes #4997

@sentrivana
sentrivana marked this pull request as ready for review August 6, 2026 10:46
@sentrivana
sentrivana requested a review from a team as a code owner August 6, 2026 10:46
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

94827 passed | ⏭️ 5502 skipped | Total: 100329 | Pass Rate: 94.52% | Execution Time: 343m 31s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -5312
Passed Tests 📉 -4334
Failed Tests 📉 -1
Skipped Tests 📉 -977

➖ Removed Tests (1)

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

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.95%    90.25%     +0.3%
==========================================
  Files          193       186        -7
  Lines        24955     23750     -1205
  Branches      9004      8594      -410
==========================================
+ Hits         22448     21435     -1013
- Misses        2507      2315      -192
- Partials      1437      1379       -58

Generated by Codecov Action

@sentrivana
sentrivana merged commit 2bd56bc into major/3.0 Aug 6, 2026
130 checks passed
@sentrivana
sentrivana deleted the ivana/major/remove-eventlet branch August 6, 2026 11:05
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