Skip to content

Remove testRunName from MacOS CI leg - #85136

Merged
JoeRobich merged 1 commit into
mainfrom
JoeRobich-patch-19
Sep 2, 2026
Merged

Remove testRunName from MacOS CI leg#85136
JoeRobich merged 1 commit into
mainfrom
JoeRobich-patch-19

Conversation

@JoeRobich

@JoeRobich JoeRobich commented Sep 2, 2026

Copy link
Copy Markdown
Member

When testRunName is given the template tries to publish xunit results. However when running Helix tests there are no results because those will be captured by the monitor job.

Microsoft Reviewers: Open in CodeFlow

When testRunName is given the template tries to publish xunit results. However when running Helix tests there are no results because those will be captured by the monitor job.
Copilot AI lite review requested due to automatic review settings September 2, 2026 21:15
@JoeRobich
JoeRobich requested a review from a team as a code owner September 2, 2026 21:15
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 2 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

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.

Copilot review overview

🟢 Approval recommended

The change is narrowly scoped to disabling an otherwise failing/irrelevant test-results publish step for the macOS Helix leg and is consistent with the template’s conditional behavior.

Review tier: Lite
Findings: None

What changed in this PR

This PR adjusts the macOS Helix test stage in Roslyn’s Azure Pipelines configuration to avoid attempting xUnit result publishing for Helix-based runs where results are handled by the Helix monitor job instead.

Changes:

  • Removed the testRunName parameter from the macOS Debug Helix test job invocation to prevent PublishTestResults@2 from running in that leg.
File Description
azure-pipelines.yml Removes testRunName from the macOS Helix test job parameters so the publish-test-results step (gated on a non-empty name) is skipped.

@JoeRobich
JoeRobich enabled auto-merge (squash) September 2, 2026 21:20
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.

4 participants