Conversation
❌AzureCLI-FullTest
|
️✔️AzureCLI-BreakingChangeTest
|
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
|
The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR. Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions). pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>
|
There was a problem hiding this comment.
Pull request overview
Updates repository CODEOWNERS to reflect recent review/ownership changes, affecting who is auto-requested for reviews across docs, tools/scripts, core, and many command modules.
Changes:
- Reassign owners for help content file patterns (
*help.py/yaml/yml). - Update ownership for various top-level paths (docs/tools/scripts/testsdk/core).
- Refresh CODEOWNERS entries for multiple
src/azure-cli/.../command_modules/*areas.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| /src/azure-cli-testsdk/ @jsntcy @jiasli @kairu-ms @wangzelin007 @bebound | ||
| /doc/ @necusjz @DanielMicrosoft @ReaNAiveD @yonzhan | ||
| /tools/ @naganandyala @bebound @YanaXu @yonzhan @wangzelin007 | ||
| /tools/aaz-flow @a0x1ab @necusjz @yanzhudd @naganandyala |
There was a problem hiding this comment.
/tools/aaz-flow is a directory, but in CODEOWNERS a pattern without a trailing / matches only that exact path, not files under it. To apply owners to all files in this directory, use /tools/aaz-flow/ (or an equivalent glob like /tools/aaz-flow/**).
| /tools/aaz-flow @a0x1ab @necusjz @yanzhudd @naganandyala | |
| /tools/aaz-flow/ @a0x1ab @necusjz @yanzhudd @naganandyala |
| /tools/ @naganandyala @bebound @YanaXu @yonzhan @wangzelin007 | ||
| /tools/aaz-flow @a0x1ab @necusjz @yanzhudd @naganandyala | ||
| /scripts/ @naganandyala @bebound @YanaXu @yonzhan @wangzelin007 | ||
| /scripts/live_test @naganandyala @bebound @YanaXu @yonzhan @wangzelin007 |
There was a problem hiding this comment.
/scripts/live_test is a directory, but without a trailing / this CODEOWNERS entry may not match files inside that directory. Use /scripts/live_test/ (or a recursive glob) so ownership applies to all files under scripts/live_test.
| /scripts/live_test @naganandyala @bebound @YanaXu @yonzhan @wangzelin007 | |
| /scripts/live_test/ @naganandyala @bebound @YanaXu @yonzhan @wangzelin007 |
Description
update the codeowners via recent reviews or ownerships.
This checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.