Skip to content

New, Set should return resource object #508

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 6 commits into from
Jun 19, 2015
Merged

New, Set should return resource object #508

merged 6 commits into from
Jun 19, 2015

Conversation

vivsriaus
Copy link
Contributor

New, Set should return resource object

@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;

@ogail
Copy link
Contributor

ogail commented Jun 17, 2015

Please update reduce the test execution time to be under 5 seconds. Right now its taking around 2 mins

@vivsriaus
Copy link
Contributor Author

@ogail I'll send the update to reduce the test execution time as a separate PR. I'm still working on it and don't want this PR to be blocked from merging because of it. Thanks

@markcowl
Copy link
Member

@vivsriaus Please fix the test run time and add comments to changelog.txt.

@markcowl
Copy link
Member

@azuresdkci retest this please

hovsepm pushed a commit that referenced this pull request Jun 19, 2015
New, Set should return resource object
@hovsepm hovsepm merged commit 495b1c4 into Azure:dev Jun 19, 2015
huangpf added a commit to AzureRT/azure-powershell that referenced this pull request Apr 20, 2016
HPF PR: dev <- Azure:dev
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.

5 participants