Weidong Xu
Weidong Xu
1. Response body has “status” and one of its possible value is “Failed”. If it is “Failed”, is it still a 200 status code (which is unusual), or 400 status...
Only review for diff from 2021-07-31-preview (and issue description) nit - inconsistent on "area" / "acreage" https://github.com/Azure/azure-rest-api-specs/blob/d9240a766d0e4bd9d245f3030a85af3f782c66d9/specification/agrifood/data-plane/Microsoft.AgFoodPlatform/preview/2022-11-01-preview/agfood.json#L1334-L1347 (several instances) my previous comment still holds, but as this is preview, you...
The rule might be intended to work for both data-plane and mgmt-plane, but its current implementation only handle for mgmt. So I assume the fix is either to make it...
OK, I will merge it ignoring failure at ModelValidation. https://github.com/Azure/azure-rest-api-specs/pull/21015#issuecomment-1269122734 But note that you still need to either suppress or fix it in 2022-09-01 branch.
@mikekistler We got approval from you and Johan, on this GA api-version. Should I proceed with the merge? I will try ignore the CI failures, as it reports on "inference.json"...
ignore Avocado ModelValidation SemanticValidation, as inference.json is OpenAPI 3.0
@msyyc @changlong-liu cosmos want to know why LRO change in commit https://github.com/Azure/azure-rest-api-specs/pull/22117/commits/c087a93f87f9f60873b37919796e9d351e07fb20 seems not reflected in Python SDK automation. (readme default tag is this preview)
@qiliao123 @tadelesh @msyyc for awareness of the breaks (that should be in preview)
@naveedaz I don't know why but I cannot add you to reviewer. (seems to be some permission problem?)