Skip to content

Conversation

@nguyenm2151
Copy link
Contributor

Related command
az acr agentpool delete

Description
Revert some changes from this PR #30195 since we found an regression. Found an issue with self.cmd not detecting a command failure in this case. #30478

Testing Guide
acr agentpool delete -n {agents_name} -r {registry_name} -y


This checklist is used to make sure that common guidelines for a pull request are followed.

Pan-Qi and others added 8 commits November 13, 2024 12:18
* add githooks

* fix src and tgt

* Add rebase check

* add rebase check

* Add rebase upstream/dev warning.

* Using merge base as target

* test

* print the warning again

* +x for hooks

* Replace echo with printf

* fix printf

* remove line for test

* Improve the code for tests

* Add docs for .githooks

* use medium min-servity for azdev linter.
* remove existing creds from test recordings

* fix azure stack

* revert unnecessary change

* fix appservice

* revert unnecessary changes for acs
@azure-client-tools-bot-prd
Copy link

Hi @nguyenm2151,
Usually we only allow pull requests to be submitted to the dev branch, please double check your pull request target branch release.

@azure-client-tools-bot-prd
Copy link

Validation for Azure CLI Full Test Starting...

Thanks for your contribution!

@azure-client-tools-bot-prd
Copy link

Validation for Breaking Change Starting...

Thanks for your contribution!

@yonzhan
Copy link
Collaborator

yonzhan commented Dec 6, 2024

Thank you for your contribution! We will review the pull request and get back to you soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot Container Registry az acr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants