Skip to content

MAINT: Use sysmon for coverage - #2636

Merged
Roman Lutz (romanlutz) merged 1 commit into
microsoft:mainfrom
romanlutz:romanlutz-faster-coverage-engine
Sep 14, 2026
Merged

MAINT: Use sysmon for coverage#2636
Roman Lutz (romanlutz) merged 1 commit into
microsoft:mainfrom
romanlutz:romanlutz-faster-coverage-engine

Conversation

@romanlutz

Copy link
Copy Markdown
Contributor

Description

Reduce coverage measurement overhead by setting COVERAGE_CORE: sysmon only on the existing CI coverage test step. The pinned coverage.py version supports this project's line coverage on Python 3.12; no branch coverage, dynamic contexts, or incompatible plugins are configured.

The measured source, four-worker loadfile scheduling, 78% overall and 90% diff coverage gates, reports, dependencies, and runner/Python versions are unchanged.

Tests and Documentation

  • Both cores passed the same 40 existing tests on Windows/Python 3.12 with four xdist workers. Actual SysMonitor selection was confirmed in all workers.
  • Executed, missing, and excluded line sets matched across all 528 files (4,365/36,848 covered lines); XML reports matched except for timestamps. The subset correctly failed the unchanged 78% gate, and boundary probes verified passing and failing behavior for both coverage gates.
  • Workflow YAML and applicable pre-commit hooks passed. Full Linux CI performance has not been measured. No documentation changes or JupyText run are needed for this workflow-only change.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@romanlutz
Roman Lutz (romanlutz) added this pull request to the merge queue Sep 14, 2026
Merged via the queue into microsoft:main with commit 80fbd43 Sep 14, 2026
41 checks passed
@romanlutz
Roman Lutz (romanlutz) deleted the romanlutz-faster-coverage-engine branch September 14, 2026 17:34
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