Skip to content

Fix Lock issue #1012

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

Merged
merged 16 commits into from
Oct 1, 2015
Merged

Fix Lock issue #1012

merged 16 commits into from
Oct 1, 2015

Conversation

vivsriaus
Copy link
Contributor

Add a test for lock
display detailed error message when template deployment fails

@azurecla
Copy link

Hi @vivsriaus, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!


It looks like you're working at Microsoft (visriniv). If you're full-time, we DON'T require a contribution license agreement.



If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@vivsriaus
Copy link
Contributor Author

@azuresdkci retest this please

@ogail
Copy link
Contributor

ogail commented Sep 30, 2015

@azuresdkci retest this please

/// <summary>
/// Gets the policy assignment.
/// </summary>
[Cmdlet(VerbsCommon.Get, "AzureRMPolicyAssignment", DefaultParameterSetName = GetAzurePolicyAssignmentCmdlet.ParameterlessSet), OutputType(typeof(PSObject))]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Name should be AzureRmXXX

@vivsriaus
Copy link
Contributor Author

@stankovski The policy files shouldn't have been included in this PR. I've deleted the files here, but will address your comments in my policy PR here: #1014

ogail added a commit that referenced this pull request Oct 1, 2015
@ogail ogail merged commit 77ee674 into Azure:dev Oct 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants