Skip to content

Sync eng/common directory with azure-sdk-tools for PR 16800 - #3350

Open
azure-sdk-automation[bot] wants to merge 1 commit into
mainfrom
sync-eng-common-djurek/codeowners-cross-cutting-exemptions-16800
Open

Sync eng/common directory with azure-sdk-tools for PR 16800#3350
azure-sdk-automation[bot] wants to merge 1 commit into
mainfrom
sync-eng-common-djurek/codeowners-cross-cutting-exemptions-16800

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#16800 See eng/common workflow

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 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.

Pull request overview

This PR syncs the repository’s eng/common infrastructure with the upstream azure-sdk-tools changes from Azure/azure-sdk-tools#16800, specifically adding Codeowners verification plumbing for Azure DevOps pipelines.

Changes:

  • Added Set-VerifyCodeownersSkip.ps1 to determine when Codeowners verification can be skipped (PR builds use PR author login; manual builds use requester email with an explicit queue-time flag).
  • Added verify-codeowners.yml pipeline steps template to run the skip evaluation and conditionally run PR diff generation plus Codeowners verification.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
eng/common/scripts/Set-VerifyCodeownersSkip.ps1 New script to compute a ShouldSkipVerifyCodeowners pipeline variable based on allowlists and build reason.
eng/common/pipelines/templates/steps/verify-codeowners.yml New pipeline steps template that wires the skip evaluation into PR/manual flows and gates verification accordingly.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread eng/common/scripts/Set-VerifyCodeownersSkip.ps1
Comment thread eng/common/scripts/Set-VerifyCodeownersSkip.ps1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants