Skip to content

Sync eng/common directory with azure-sdk-tools for PR 16726 - #10231

Merged
azure-sdk-automation[bot] merged 2 commits into
mainfrom
sync-eng-common-users/raych1/resolve-pkg-update-16726
Aug 14, 2026
Merged

Sync eng/common directory with azure-sdk-tools for PR 16726#10231
azure-sdk-automation[bot] merged 2 commits into
mainfrom
sync-eng-common-users/raych1/resolve-pkg-update-16726

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

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

@azure-sdk-automation
azure-sdk-automation Bot requested review from Ray Chen (raych1) and a balanced review from Copilot August 13, 2026 22:23
@azure-sdk-automation
azure-sdk-automation Bot requested a review from a team as a code owner August 13, 2026 22:23
@azure-sdk-automation azure-sdk-automation Bot added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Aug 13, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.
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

Note

Copilot could not run the full agentic suite for this review because it was automatically requested on a bot-authored pull request. Request a review from Copilot under Reviewers to retry with the full agentic suite. Improved support for bot-authored pull requests is coming soon.

Updates auto-release package resolution to avoid treating service-root file edits as service-wide changes, preventing unintended broad package resolution in sdk/<service>/ layouts.

Changes:

  • Add -ExcludeServiceRootFiles switch to New-GitHubPullRequestDiffObject to filter out files directly under sdk/<service>/.
  • Enable this exclusion when building the PR diff in Resolve-AutoReleasePackages.ps1.
  • Update inline documentation to clarify the new exclusion behavior.

Reviewed changes

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

File Description
eng/common/scripts/Resolve-AutoReleasePackages.ps1 Passes the new exclusion switch when constructing the PR diff to narrow package detection scope.
eng/common/scripts/AutoRelease-Operations.ps1 Implements -ExcludeServiceRootFiles filtering in diff construction and documents the new parameter.

💡 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/AutoRelease-Operations.ps1
Comment thread eng/common/scripts/AutoRelease-Operations.ps1
@azure-sdk-automation
azure-sdk-automation Bot merged commit 286968e into main Aug 14, 2026
9 checks passed
@azure-sdk-automation
azure-sdk-automation Bot deleted the sync-eng-common-users/raych1/resolve-pkg-update-16726 branch August 14, 2026 16:03
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.

2 participants