Skip to content

Remove cosmos filter from pullrequest build definition#46425

Open
scbedd wants to merge 4 commits intomainfrom
test-cosmos-triggering
Open

Remove cosmos filter from pullrequest build definition#46425
scbedd wants to merge 4 commits intomainfrom
test-cosmos-triggering

Conversation

@scbedd
Copy link
Copy Markdown
Member

@scbedd scbedd commented Apr 20, 2026

Remove sdk/cosmos/ exclusion from pullrequest.yml trigger paths and ExcludePaths parameter so cosmos changes go through the standard PR validation pipeline.

A dummy .trigger-test file is included under sdk/cosmos/ to ensure the PR pipeline triggers for cosmos paths.

Related to Azure/azure-sdk-tools#15160

Remove sdk/cosmos/ exclusion from pullrequest.yml trigger paths
and ExcludePaths parameter so cosmos changes go through the standard
PR validation pipeline.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 20, 2026 20:47
@scbedd scbedd requested review from a team, benbp and weshaggard as code owners April 20, 2026 20:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 updates the repo’s PR validation pipeline configuration so changes under sdk/cosmos/ are no longer filtered out and will run through the standard PR validation pipeline.

Changes:

  • Removed sdk/cosmos/ from eng/pipelines/pullrequest.yml PR path exclusions.
  • Removed the corresponding ExcludePaths template parameter entry that duplicated the PR path exclusion list.
  • Added sdk/cosmos/.trigger-test as a sentinel file to validate that cosmos-path PR changes will trigger the pipeline.
Show a summary per file
File Description
eng/pipelines/pullrequest.yml Removes the cosmos-specific PR trigger exclusion and the duplicated ExcludePaths exclusion list so cosmos changes are validated by the standard PR pipeline.
sdk/cosmos/.trigger-test Adds a no-op file under sdk/cosmos/ to exercise PR path triggering for cosmos changes.

Copilot's findings

  • Files reviewed: 1/2 changed files
  • Comments generated: 0

@scbedd
Copy link
Copy Markdown
Member Author

scbedd commented Apr 20, 2026

Failing cosmos checks are unrelated to this change. Going to bypass and merge.

@scbedd
Copy link
Copy Markdown
Member Author

scbedd commented Apr 20, 2026

/check-enforcer override

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

Labels

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants