-
Notifications
You must be signed in to change notification settings - Fork 4k
Websites: Add new snapshots cmdlets #5996
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
Conversation
Recorded snapshots tests again Regenerated Restore-AzureRmWebAppSnapshot and filled in help examples. Refactored RestoreAzureWebAppSnapshot cmdlet. ResourceGroup and Name parameters specify target app. Rewrote RestoreAzureRmWebAppSnapshot cmdlet help.
@Nking92 You will need to fix the issues outlined here: https://azuresdkci.westus2.cloudapp.azure.com/job/powershell/4012/artifact/src/Package/ValidateHelpIssues.csv pull the latest, and fix merge conflicts |
@@ -300,7 +300,6 @@ | |||
<Service Include="{82A7F48D-3B50-4B1E-B82E-3ADA8210C358}" /> | |||
</ItemGroup> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to add your new recorded mock files here and ensure they are set to CopyAlways
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the reason for the ci failure
Description
Adds two new cmdlets. Get-AzureRmWebAppSnapshot returns the snapshots available for a web app. Restore-AzureRmWebAppSnapshot restores a snapshot to a web app.
This branch is rebased on top of pending PR #5967 because it needs the new SDK.
Checklist
CONTRIBUTING.md
platyPS
module