[Maps] az maps account create: Add default value for --sku#32711
[Maps] az maps account create: Add default value for --sku#32711
az maps account create: Add default value for --sku#32711Conversation
️✔️AzureCLI-FullTest
|
|
| rule | cmd_name | rule_message | suggest_message |
|---|---|---|---|
| maps account create | cmd maps account create update parameter name: removed property required=True |
||
| maps account update | cmd maps account update update parameter name: removed property required=True |
|
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
This PR adds a default value of 'G2' for the --sku parameter in the az maps account create command, making it optional for users since Azure Maps now only supports this SKU for new accounts.
Changes:
- Made
--skuparameter optional by changingrequired=Truetorequired=Falsein parameter definitions - Added default value logic to use 'G2' when
--skuis not specified - Update command remains unaffected as it already handles optional SKU parameter correctly
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/azure-cli/azure/cli/command_modules/maps/_params.py | Changed --sku parameter from required to optional for maps account commands |
| src/azure-cli/azure/cli/command_modules/maps/custom.py | Added default value 'G2' for SKU in create function when parameter is not provided |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
az maps account create Add default value for --skuaz maps account create: Add default value for --sku
|
better to have a test case to cover it. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
Related command
az maps account create
Description
Azure Maps now only supports one
--sku(G2) for new accounts. The CLI should not require users to provide a --sku when creating new accounts.Testing Guide
Manually verified default value used when --sku is omitted. No change to test recordings resulting form this change.
History Notes
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.