Skip to content

[DO NOT MERGE] [NO REVIEW] Run all build legs#128477

Draft
premun wants to merge 2 commits into
dotnet:mainfrom
premun:prvysoky/run-all-legs
Draft

[DO NOT MERGE] [NO REVIEW] Run all build legs#128477
premun wants to merge 2 commits into
dotnet:mainfrom
premun:prvysoky/run-all-legs

Conversation

@premun

@premun premun commented May 22, 2026

Copy link
Copy Markdown
Member

Copilot AI review requested due to automatic review settings May 22, 2026 11:21
@premun premun added NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it labels May 22, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

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.

Pull request overview

This PR changes the evaluate-changed-paths.sh pipeline helper to always report “contains change” by unconditionally setting the Azure DevOps output variable to true, bypassing the include/exclude path diff evaluation logic.

Changes:

  • Adds an unconditional early-return in probePaths() that sets the pipeline variable to true for every subset.
  • Skips all subsequent exclude/include/combined probing and matched-path reporting.

Comment thread eng/pipelines/evaluate-changed-paths.sh
@premun

premun commented Jun 4, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

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

Labels

area-Infrastructure NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants