Tim Lovell-Smith
Tim Lovell-Smith
> "default": "None", You removed the documented Default value of 'None', but added "If not specified, the default is none.". Is this intended as a bugfix, a breaking change, or...
> "title": "Audit / Enforce", doesn't seem like the right title for the mode property. Do you want something like "Mode: audit vs enforce"? --- Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2025-06-01/BatchManagement.json:6736 in 13f5ea6....
Nothing mandatory to fix at this point for me, but please review remaining feedback/questions.
You should fix or suppress the remaining lintdiff warnings introduced in this PR: ⚠️ SchemaDescriptionOrTitle | Schema should have a description or title.Location: Microsoft.Batch/stable/2025-06-01/BatchManagement.json#L5614 | -- | -- | --...
Since this is a new version, consider adding a provisioningState to your model to fix this historic linter warning >201 response schema in long running PUT operation is missing ProvisioningState...
Since this is a new version, you should consider fixing this historic issue: > ❌ PutRequestResponseSchemeArm | A PUT operation request body schema should be the same as its 200...
Since this is a new api version, consider relaxing some of the 'required' constraints for PATCH requests: > ❌ PatchBodyParametersSchema | Properties of a PATCH request body must not be...