-
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
Add Database Migration Service sdk for public preview #3867
Conversation
…nges. Changes: - Update tests using Azure Resource Manager 1.6.0-preview and other changes. - Update sdk models based on latest swagge in 'current' branch in 'azure-rest-spec-api-pr' - Add _metadata folder created by running generate.cmd
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.
Everything looks good. Good to merge after addressing minor feedback given.
<ItemGroup> | ||
<PackageReference Include="Microsoft.Azure.KeyVault" Version="2.3.2" /> | ||
<PackageReference Include="Microsoft.Azure.Management.ResourceManager" Version="1.6.0-preview" /> | ||
<PackageReference Update="Microsoft.NET.Test.Sdk" Version="15.3.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.
@hitenjava Microsoft.NET.Test.Sdk and Rest.ClientRuntime.Azure.TestFramework references are implicitly added to your project, so you don't have to explicitlty add them.
When we change these packages or upgrade them, your project will be automatically upgraded at the same time.
Having such explicit references inhibit us to upgrade repo wide such changes.
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.
Fixed.
@hitenjava Update the full Azure github path to the swagger in the summary: https://github.com/Azure/azure-rest-api-specs/tree/current/specification/datamigration/resource-manager/Microsoft.DataMigration/2017-11-15-preview |
…estFramework references.
Link to swagger here -
https://github.com/Azure/azure-rest-api-specs/tree/current/specification/datamigration/resource-manager/Microsoft.DataMigration/2017-11-15-preview
Description
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.