Chris Stackhouse
Chris Stackhouse
@zaowang-ms - If these changes were already reviewed and signed off by ARM in the private repo, just provide a link to the PR and I'll wave it through. We...
@zaowang-ms - Signing off for ARM per signoff in the private repo. For future APIs, please use verbs for POST actions.
/azp run
> @mentat9 I still have some concerns about the verb thing. Waving it through means we'll never be able to fix it to a verb without it being a breaking...
@xuhumsft - Couple of things needed before ARM review: 1. Complete the intake form (Due diligence checklist). 2. Address this comment: https://github.com/Azure/azure-rest-api-specs/pull/27214#issuecomment-1872433306. Since one of the changes is to rename...
@tadelesh, @hsurana06 - Please address the NotReadyForARMReview issues described in Next Steps to Merge next. This needs to happen before ARM review.
@tadelesh, @hsurana06 - Your PR is not ready for ARM review. Don't remove that label yourself: it's handled by the automation. This comment is your guide: https://github.com/Azure/azure-rest-api-specs/pull/26644#issuecomment-1803829331.
/azp run
@allenjzhang, @konrad-jamrozik - Does this PR actually need review by ARM? I see @visingla-ms left a couple of unaddressed comments. If it doesn't require review, please keep it out of...