Skip to content

Conversation

@luluRagdoll
Copy link

@luluRagdoll luluRagdoll commented Sep 4, 2018

Description

Add CosmosDb in reserved resource enum type.
Add name property in PatchProperties

Checklist

Maddie Clayton and others added 3 commits August 30, 2018 15:23
Update AzureStack.Compute.Admin powershell quota object to support two additional managed disk parameters
…pe enum. Add name property in the PatchProperties
@luluRagdoll
Copy link
Author

Working on resolving the CI failure

@deathly809
Copy link
Member

@luluRagdoll Why did you add me and Bala as reviewers? Is this in Azure Stack?

@luluRagdoll
Copy link
Author

You two were suggested as the reviewers in the PR reviewer section after I submitted this PR.

@deathly809
Copy link
Member

@luluRagdoll We only review for Azure Stack. I removed us, you should add @maddieclayton/ @cormacpayne/@MiYanni

@maddieclayton
Copy link
Contributor

@luluRagdoll Not sure how, but looks like you picked up a bunch of changes from Stack. Can you rebase so only your changes are added?

@maddieclayton
Copy link
Contributor

@luluRagdoll please link your cmdlet design review from here: https://github.com/Azure/azure-powershell-cmdlet-review-pr

@maddieclayton
Copy link
Contributor

Closing in favor of: #7164

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants