Weidong Xu
Weidong Xu
@raosuhas All PR (with change to API path) merging into this feature branch has been approved by ARM reviewer - https://github.com/Azure/azure-rest-api-specs/pull/21015 (by Tim) - https://github.com/Azure/azure-rest-api-specs/pull/22159 - https://github.com/Azure/azure-rest-api-specs/pull/22211 - https://github.com/Azure/azure-rest-api-specs/pull/22491
@MaryGao Please check JS SDK breaking changes https://github.com/Azure/azure-rest-api-specs/pull/22084/checks?check_run_id=11705120988
@msyyc on python failure.
> We will fix the failure soon and please ignore it temporarily. Sure. I will probably ask Arthur to force merge this tomorrow.
depends on the scale of the changes, you may need schedule a review https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/271/Azure-REST-API-Stewardship
> This PR is based on #20245. It only changes one of the parameters from 'location' to 'coordinate' in /search/polygon and it looks good. approved in https://github.com/Azure/azure-rest-api-specs/pull/20245 by Mark and...
Would you link the issue of the review (the issue)? So I can check the log.
@mikekistler I took another look and approved. Added 2 comments. One "nit" that might affect codegen/SDK but won't block REST API. Another about whether we had `result` in action status...
First and foremost, please make your CI pass.
@pjohari-ms Regardless of whether it is caused by your change or not, all of them is required to be fix or at least suppressed. Some of them are new rules...