Skip to content

[azsdk-cli] Revert to using azp for PR CI#12074

Merged
benbp merged 1 commit into
Azure:mainfrom
benbp:benbp/azsdk-cli-pipeline-ci
Sep 17, 2025
Merged

[azsdk-cli] Revert to using azp for PR CI#12074
benbp merged 1 commit into
Azure:mainfrom
benbp:benbp/azsdk-cli-pipeline-ci

Conversation

@benbp

@benbp benbp commented Sep 15, 2025

Copy link
Copy Markdown
Member

The github action has slowed down a bit with newer changes so we're going to revert to focusing on performance improvements in the azure pipelines CI instead.

Copilot AI review requested due to automatic review settings September 15, 2025 18:42
@benbp benbp added the Central-EngSys This issue is owned by the Engineering System team. label Sep 15, 2025
@benbp
benbp requested a review from scbedd as a code owner September 15, 2025 18:42
@benbp benbp self-assigned this Sep 15, 2025
@benbp
benbp requested a review from weshaggard September 15, 2025 18:42

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 disables pull request validation for the azsdk-cli Azure DevOps pipeline by removing the PR trigger configuration and replacing it with pr: none. This allows the pipeline to continue running on main branch triggers while skipping CI validation for pull requests targeting this tool.

  • Removes PR branch and path triggers for the azsdk-cli pipeline
  • Replaces multi-line PR configuration with pr: none to disable PR validation

@benbp benbp moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 🚀🌒🧑‍🚀 Sep 15, 2025
@benbp benbp added the azsdk-cli Issues related to Azure/azure-sdk-tools::tools/azsdk-cli label Sep 15, 2025
@weshaggard

Copy link
Copy Markdown
Member

Let's talk about this a little more tomorrow in our 1:1. While I appreciate GH actions might spin up faster I worry that things will get out-of-sync with what we ship and what we validate.

@benbp
benbp force-pushed the benbp/azsdk-cli-pipeline-ci branch from 2defe23 to e81e60c Compare September 17, 2025 17:15
@benbp benbp changed the title [azsdk-cli] Skip azp ci [azsdk-cli] Revert to using azp for PR CI Sep 17, 2025
@benbp
benbp merged commit a667bcd into Azure:main Sep 17, 2025
2 checks passed
@benbp
benbp deleted the benbp/azsdk-cli-pipeline-ci branch September 17, 2025 17:16
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 🚀🌒🧑‍🚀 Sep 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

azsdk-cli Issues related to Azure/azure-sdk-tools::tools/azsdk-cli Central-EngSys This issue is owned by the Engineering System team.

Projects

Development

Successfully merging this pull request may close these issues.

5 participants