Skip to content

Sync eng/common directory with azure-sdk-tools for PR 10858#36098

Merged
azure-sdk merged 1 commit intomainfrom
sync-eng/common-copilot/fix-10836-2-10858
Oct 2, 2025
Merged

Sync eng/common directory with azure-sdk-tools for PR 10858#36098
azure-sdk merged 1 commit intomainfrom
sync-eng/common-copilot/fix-10836-2-10858

Conversation

@azure-sdk
Copy link
Collaborator

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

Co-authored-by: benbp <1020379+benbp@users.noreply.github.com>
@azure-sdk azure-sdk requested a review from a team as a code owner October 1, 2025 19:41
Copilot AI review requested due to automatic review settings October 1, 2025 19:41
@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 Oct 1, 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 synchronizes the eng/common directory with azure-sdk-tools repository for PR 10858, updating pipeline template conditions to include succeeded() checks. This ensures pipeline steps only execute when previous steps have completed successfully.

Key Changes

  • Added succeeded() condition checks to Azure DevOps pipeline templates
  • Updated conditions in API change detection and performance job templates

Reviewed Changes

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

File Description
eng/common/pipelines/templates/steps/detect-api-changes.yml Added succeeded() condition to API change detection step
eng/common/pipelines/templates/jobs/perf.yml Added succeeded() condition to profile artifact publishing step

@azure-sdk azure-sdk merged commit 28f213b into main Oct 2, 2025
17 checks passed
@azure-sdk azure-sdk deleted the sync-eng/common-copilot/fix-10836-2-10858 branch October 2, 2025 20:21
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.

4 participants