Skip to content

Skip coverage collection on Windows CI - #11976

Open
Timothee Guerin (timotheeguerin) with Copilot wants to merge 2 commits into
mainfrom
copilot/disable-coverage-collection-windows-ci
Open

Timothee Guerin (timotheeguerin) with Copilot wants to merge 2 commits into
mainfrom
copilot/disable-coverage-collection-windows-ci

Conversation

Copilot AI commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Windows CI was spending extra time collecting and uploading coverage that is already produced by non-Windows jobs. This narrows coverage work to non-Windows runners while keeping Windows test coverage for behavior intact.

  • Core CI tests

    • Run Vitest without --coverage on Windows matrix jobs.
    • Keep coverage-enabled Vitest runs on Linux/macOS-style runners.
  • Coverage artifacts

    • Skip coverage artifact upload for Windows jobs.
    • Preserve existing artifact naming and retention for non-Windows jobs.
- name: Test
  if: runner.os == 'Windows'
  run: pnpm vitest run --reporter=default --reporter=github-actions

- name: Test with coverage
  if: runner.os != 'Windows'
  run: pnpm vitest run --coverage --reporter=default --reporter=github-actions

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Comment thread .github/workflows/core-ci.yml Outdated
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>

Copilot AI 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.

🟢 Approval recommended

No unresolved review issues were identified.

Pull request overview

Updates Core CI to skip Vitest coverage collection and artifact uploads on Windows while preserving coverage on non-Windows runners.

Changes:

  • Adds OS-specific coverage settings.
  • Skips coverage artifact uploads for Windows jobs.
  • Preserves non-Windows coverage runs and artifacts.
File summaries
File Description
.github/workflows/core-ci.yml Configures OS-specific Vitest coverage and artifact behavior.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@azure-sdk-automation

Copy link
Copy Markdown

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants