Skip to content

Conversation

@soeom
Copy link
Contributor

@soeom soeom commented Apr 10, 2020

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@soeom soeom changed the title [CosmosDB]Support az cosmosdb private-link-resource and private-endpoint [CosmosDB]Support az cosmosdb private-link-resource and private-endpoint-connection Apr 10, 2020
@yonzhan yonzhan added this to the S168 milestone Apr 11, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented Apr 11, 2020

add to S168

@soeom
Copy link
Contributor Author

soeom commented Apr 15, 2020

@jsntcy @qianwens @myronfanqiu @Juliehzl Can you review this PR?

Copy link
Contributor

@Juliehzl Juliehzl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix all comments I left and take a look for command guideline here to check again.

@soeom soeom requested review from Juliehzl and mmyyrroonn April 16, 2020 00:20
@soeom
Copy link
Contributor Author

soeom commented Apr 16, 2020

@myronfanqiu @Juliehzl I fixed following all comments and command guideline. Please take a look at PR again.

Copy link
Contributor

@Juliehzl Juliehzl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. But please make CI pass.

@soeom
Copy link
Contributor Author

soeom commented Apr 17, 2020

Please help merge the PR @myronfanqiu

@soeom soeom requested a review from mmyyrroonn April 17, 2020 01:01
@mmyyrroonn mmyyrroonn merged commit 1a803c4 into Azure:dev Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants