Skip to content

Conversation

@eshaparmar
Copy link
Contributor

*We have added new parameters/functionality for the DSC Extension - Publish-AzureVMDscConfiguration cmdlet (both arm/asm).

*Tests for these are written in powershell in our code base and run through our daily CI system

@azurecla
Copy link

Hi @eshaparmar, 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 (esparmar). 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;

@eshaparmar
Copy link
Contributor Author

@Azure/adx-sdk-team

@stankovski
Copy link
Member

@azuresdkci retest this please

@yugangw-msft
Copy link
Contributor

please resolve the confliction

Conflicts:
	src/Common/Commands.Common/Properties/Resources.resx
@stankovski
Copy link
Member

Copy link
Contributor

Choose a reason for hiding this comment

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

license header

@yugangw-msft
Copy link
Contributor

@eshaparmar
Copy link
Contributor Author

I talked to Denis offline about it. Those tests are working locally but for some reason they fail now on build system. I have disabled those tests temporarily. Please merge this PR

@yugangw-msft yugangw-msft changed the title New functionality for DSC Extension - Publish-AzureVMDscConfiguration cmdlet New functionality for DSC Extension - Publish-AzureVMDscConfiguration cmdlet(have merge conflicts) Aug 1, 2015
@eshaparmar
Copy link
Contributor Author

Closing this PR and created a new one #689

@eshaparmar eshaparmar closed this Aug 3, 2015
@eshaparmar eshaparmar deleted the publish-newfunctionality branch August 12, 2015 15:36
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