Skip to content

Conversation

@ritwikbasu
Copy link
Contributor

@ritwikbasu ritwikbasu commented Jun 26, 2018

Description

Checklist

@ritwikbasu
Copy link
Contributor Author

This is a retake from #6277 which was dependent on SDK version update for Network

@ritwikbasu ritwikbasu requested a review from maddieclayton June 26, 2018 21:35
@MikhailTryakhov
Copy link
Contributor

@ritwikbasu please cover these properties by unit tests

@ritwikbasu
Copy link
Contributor Author

@MikhailTryakhov - Since these are new values added to existing enums, the existing scenario tests already present still hold. There is no change in any business logic or commandlet design.

@MikhailTryakhov
Copy link
Contributor

LGTM, @praries880 could you please help to merge?

@praries880 praries880 self-assigned this Jun 27, 2018
@praries880 praries880 changed the base branch from Network-2018-06-01 to release-2018-06-29 June 27, 2018 17:48
@praries880
Copy link
Contributor

@ritwikbasu kindly take care of the merge conflicts

@praries880
Copy link
Contributor

@ritwikbasu the changes look good to me, just take care of the merge conflicts n we'r good to go

@ritwikbasu
Copy link
Contributor Author

@praries880 - I have resolved the merge conflicts. Please let me know if everything looks good.

@praries880
Copy link
Contributor

@MikhailTryakhov
Copy link
Contributor

@praries880 can we merge it?

@praries880 praries880 merged commit 7258a45 into Azure:release-2018-06-29 Jun 27, 2018
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.

7 participants