-
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
Update Cosmos DB SDK to 06-01-preview #15220
Conversation
Co-authored-by: Meha Kaushik <mekaushi@microsoft.com>
…nto kristynh-new
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.
Could you:
- Check in the metadata TXT file generated when you run
generate.ps1
(details)
Everything else looks good. Thanks
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
...mosdb/Microsoft.Azure.Management.CosmosDB/tests/ScenarioTests/SqlResourcesOperationsTests.cs
Outdated
Show resolved
Hide resolved
...mosdb/Microsoft.Azure.Management.CosmosDB/tests/ScenarioTests/SqlResourcesOperationsTests.cs
Show resolved
Hide resolved
...mosdb/Microsoft.Azure.Management.CosmosDB/tests/ScenarioTests/SqlResourcesOperationsTests.cs
Show resolved
Hide resolved
...mosdb/Microsoft.Azure.Management.CosmosDB/tests/ScenarioTests/SqlResourcesOperationsTests.cs
Show resolved
Hide resolved
...mosdb/Microsoft.Azure.Management.CosmosDB/tests/ScenarioTests/SqlResourcesOperationsTests.cs
Show resolved
Hide resolved
@nichatur Thank you for your suggestions. I have updated the tests. Please review, thanks! |
...mosdb/Microsoft.Azure.Management.CosmosDB/tests/ScenarioTests/SqlResourcesOperationsTests.cs
Outdated
Show resolved
Hide resolved
Hi @allenjzhang I have updated the metadata txt file. Thank you! |
Update Cosmos DB SDK from 2020-04-01 to 2020-06-01-preview
This change includes added Sql RBAC operations + test coverage