-
Notifications
You must be signed in to change notification settings - Fork 413
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
chore(ci_visibility): add threading and multiprocessing coverage tests #9479
chore(ci_visibility): add threading and multiprocessing coverage tests #9479
Conversation
undo accidental riot fix.
Co-authored-by: Gabriele N. Tornetta <P403n1x87@users.noreply.github.com>
BenchmarksBenchmark execution time: 2024-06-11 10:11:32 Comparing candidate commit 638baa8 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 221 metrics, 9 unstable metrics. |
…com:DataDog/dd-trace-py into romain.komorn/CIVIS-9428/add_coverage_tests
Datadog ReportBranch report: ✅ 0 Failed, 119008 Passed, 59251 Skipped, 3h 57m 54.03s Total duration (3h 32m 0.93s time saved) |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9479 +/- ##
===========================================
- Coverage 75.89% 10.27% -65.62%
===========================================
Files 1315 1285 -30
Lines 124966 123124 -1842
===========================================
- Hits 94837 12651 -82186
- Misses 30129 110473 +80344 ☔ View full report in Codecov by Sentry. |
Adds some tests for the internal coverage tool.
Also fixes a bug where coverage absorbed from threading or multiprocessing modules was being added to the session-level coverage even when said coverage wasn't enabled.
No release note because this is experimental and unreleased.
Checklist
changelog/no-changelog
is set@DataDog/apm-tees
.Reviewer Checklist