-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Generate SDK for new API version #5151
Conversation
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.
Was this code generated using a generate.ps1 script?
@@ -6,17 +6,19 @@ | |||
<PropertyGroup> | |||
<PackageId>Microsoft.Azure.Management.Blueprint</PackageId> | |||
<Description>Provides Microsoft Azure Blueprint management operations including the ability to create, update and delete Blueprint definition and assignment.</Description> | |||
<Version>0.9.1-Preview</Version> | |||
<Version>0.10.0-Preview</Version> |
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.
Since this is a new API version, the Nuget major version needs to be bumped too.
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.
@filizt As we discussed offline, please bump this version to 1.8.0-preview
(please note the casing)
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.
@dsgouda after out email thread, updating the version as 0.10.0-preview
src/SDKs/Blueprint/Management.Blueprint/Microsoft.Azure.Management.Blueprint.csproj
Show resolved
Hide resolved
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.
LGTM
Will merge on CIs passing azure-sdk-for-net.client can be ignored |
Description
Blueprint API change PRs
Azure/azure-rest-api-specs#4590
Azure/azure-rest-api-specs#4945
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines
Testing Guidelines
SDK Generation Guidelines
*.csproj
andAssemblyInfo.cs
files have been updated with the new version of the SDK.Blueprint API change PRs
Azure/azure-rest-api-specs#4590
Azure/azure-rest-api-specs#4945