Skip to content

Sync eng/common directory with azure-sdk-tools for PR 14661#37784

Merged
azure-sdk merged 5 commits into
mainfrom
sync-eng/common-djurek/codeowners-block-edit-14661
Mar 24, 2026
Merged

Sync eng/common directory with azure-sdk-tools for PR 14661#37784
azure-sdk merged 5 commits into
mainfrom
sync-eng/common-djurek/codeowners-block-edit-14661

Conversation

@azure-sdk
Copy link
Copy Markdown
Collaborator

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

@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Mar 23, 2026
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

Syncs eng/common with updates from azure-sdk-tools PR 14661 by adding a pipeline check that prevents protected .github/CODEOWNERS sections from being modified in normal PRs.

Changes:

  • Added a PowerShell script to export and compare named CODEOWNERS sections using the azsdk CLI.
  • Added a pipeline step template to detect CODEOWNERS changes in PRs and run the protected-section comparison.
  • Extended the install-azsdk-cli.yml step template to accept a caller-provided condition.

Reviewed changes

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

File Description
eng/common/scripts/Test-CodeownersSections.ps1 New script that exports and diffs specified CODEOWNERS sections via azsdk config codeowners export-section.
eng/common/pipelines/templates/steps/verify-codeowners-sections.yml New PR-only pipeline step template that prepares before/after CODEOWNERS files and runs the section comparison.
eng/common/pipelines/templates/steps/install-azsdk-cli.yml Adds a Condition parameter so callers can control when the azsdk CLI install runs.

Comment thread eng/common/pipelines/templates/steps/verify-codeowners-sections.yml
Comment thread eng/common/scripts/Test-CodeownersSections.ps1
@azure-sdk azure-sdk force-pushed the sync-eng/common-djurek/codeowners-block-edit-14661 branch from 51b32f6 to e9bc547 Compare March 24, 2026 03:54
@azure-sdk azure-sdk requested a review from a team as a code owner March 24, 2026 03:54
@azure-sdk azure-sdk force-pushed the sync-eng/common-djurek/codeowners-block-edit-14661 branch from e9bc547 to 6d333a0 Compare March 24, 2026 04:47
@azure-sdk azure-sdk force-pushed the sync-eng/common-djurek/codeowners-block-edit-14661 branch from 6d333a0 to 3b42ae4 Compare March 24, 2026 04:58
@azure-sdk azure-sdk merged commit 1d5bfc0 into main Mar 24, 2026
13 checks passed
@azure-sdk azure-sdk deleted the sync-eng/common-djurek/codeowners-block-edit-14661 branch March 24, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants