Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ARM] az bicep restore: Add new command to restore external modules #22423

Merged
merged 3 commits into from May 18, 2022

Conversation

shenglol
Copy link
Contributor

@shenglol shenglol commented May 12, 2022

Related command
az bicep

Description
bicep restore was added to Bicep CLI since v0.4.1008. The PRs adds a new command to support it. It also adds a few parameters to other Bicep command to keep az bicep and Bicep CLI in sync.

Closes Azure/bicep#6618.
Closes Azure/bicep#6350.

Testing Guide

az bicep restore -f main.bicep
az bicep restore -f main.bicep --force

History Notes

[ARM] az bicep restore: Add new command to restore external modules
[ARM] az bicep build: Add new parameter --no-restore to allow compilation without restoring external modules
[ARM] az bicep decompile: Add new parameter --force to allow overwriting existing Bicep files


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

@ghost ghost added the Auto-Assign Auto assign by bot label May 12, 2022
@ghost ghost requested a review from yonzhan May 12, 2022 23:57
@ghost ghost assigned zhoxing-ms May 12, 2022
@ghost ghost added this to the May 2022 (2022-05-24) - For Build milestone May 12, 2022
@ghost ghost added the ARM az resource/group/lock/tag/deployment/policy/managementapp/account management-group label May 12, 2022
@shenglol shenglol changed the title [ARM ] az bicep restore: Add new command to restore external modules [ARM] az bicep restore: Add new command to restore external modules May 12, 2022
@yonzhan
Copy link
Collaborator

yonzhan commented May 13, 2022

ARM

@zhoxing-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@zhoxing-ms
Copy link
Contributor

zhoxing-ms commented May 13, 2022

@shenglol Could we add some tests for those new commands?

@shenglol
Copy link
Contributor Author

Sure. Added live scenario tests for the new commands and parameters.

@zhoxing-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@wangzelin007
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@zhoxing-ms zhoxing-ms merged commit 215960c into Azure:dev May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ARM az resource/group/lock/tag/deployment/policy/managementapp/account management-group Auto-Assign Auto assign by bot
Projects
None yet
5 participants