Skip to content

Sync eng/common directory with azure-sdk-tools for PR 10657#8853

Merged
scbedd merged 2 commits intomainfrom
sync-eng/common-adjust-services-10657
May 16, 2025
Merged

Sync eng/common directory with azure-sdk-tools for PR 10657#8853
scbedd merged 2 commits intomainfrom
sync-eng/common-adjust-services-10657

Conversation

@azure-sdk
Copy link
Collaborator

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

@azure-sdk azure-sdk requested review from Copilot and scbedd May 16, 2025 18:28
@azure-sdk azure-sdk requested a review from a team as a code owner May 16, 2025 18:28
@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 May 16, 2025
Copy link
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 Azure DevOps pipeline to sync the eng/common directory with recent changes from azure-sdk-tools, specifically enhancing how package properties are saved and processed for pull requests.

  • Introduces a new Powershell@2 task to save package properties filtered for PR execution.
  • Adds an inline PowerShell script to read and filter JSON package files to determine the changed services.
  • Removes a redundant save-package-properties step.

@azure-sdk azure-sdk force-pushed the sync-eng/common-adjust-services-10657 branch from b092bc0 to a17c980 Compare May 16, 2025 19:44
@scbedd scbedd merged commit 838e048 into main May 16, 2025
4 checks passed
@scbedd scbedd deleted the sync-eng/common-adjust-services-10657 branch May 16, 2025 21:29
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