Tim Lovell-Smith

Results 377 comments of 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...