-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
update description for all properties, new autorest client and vnet get on vnet peering from vnet #2621
update description for all properties, new autorest client and vnet get on vnet peering from vnet #2621
Conversation
Hi @DeepakRajendranMsft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution! TTYL, AZPRBOT; |
@@ -1,5 +1,5 @@ | |||
{ | |||
"version": "8.1.0-preview", | |||
"version": "9.0.0-preview", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DeepakRajendranMsft is there a breaking change in this PR? If you are just regenerating the code based on a newer version of AutoRest, you should bump the version to 8.1.1-preview
@@ -19,8 +19,8 @@ | |||
[assembly: AssemblyTitle("Microsoft Azure Network Management Library")] | |||
[assembly: AssemblyDescription("Provides Microsoft Azure Network management functions for managing the Microsoft Azure Network service.")] | |||
|
|||
[assembly: AssemblyVersion("8.1.0.0")] | |||
[assembly: AssemblyFileVersion("8.1.0.0")] | |||
[assembly: AssemblyVersion("9.0.0.0")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DeepakRajendranMsft same comment about version number
@@ -214,5 +214,160 @@ public void VirtualNetworkCheckIpAddressAvailabilityTest() | |||
networkManagementClient.VirtualNetworks.Delete(resourceGroupName, vnetName); | |||
} | |||
} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DeepakRajendranMsft what new functionality are you trying to test here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed offline
This is a continuation of #2530. I have generated the sdk with the latest autorest client
@cormacpayne @markcowl