Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates HealthDataAiService.Deid with feedback from SDK Review Team #29036

Open
wants to merge 32 commits into
base: healthdataaiservices-deidentification-2024-01-16-preview
Choose a base branch
from

Convert to diff long running operation

b6fd5ea
Select commit
Failed to load commit list.
Open

Updates HealthDataAiService.Deid with feedback from SDK Review Team #29036

Convert to diff long running operation
b6fd5ea
Select commit
Failed to load commit list.
openapi-pipeline-app / Swagger LintDiff succeeded May 13, 2024 in 3m 24s

Swagger LintDiff succeeded

Details

Compared specs (v2.2.2) new version base version
package-2024-01-16-preview package-2024-01-16-preview(b6fd5ea) default(healthdataaiservices-deidentification-2024-01-16-preview)

[must fix]The following errors/warnings are introduced by current PR:

Rule Message Related RPC [For API reviewers]
鈿狅笍 PutRequestResponseScheme A PUT operation request body schema should be the same as its 200 response schema, to allow reusing the same entity between GET and PUT. If the schema of the PUT request body is a superset of the GET response body, make sure you have a PATCH operation to make the resource updatable. Operation: 'Deidentification_CreateJob' Request Model: 'parameters[3].schema' Response Model: 'responses[200].schema'
Location: HealthDataAIServices.Deidentification/preview/2024-01-16-preview/HealthDataAIServices.Deidentification.json#L213
鈿狅笍 OperationId OperationId for put method should contain both 'Create' and 'Update'
Location: HealthDataAIServices.Deidentification/preview/2024-01-16-preview/HealthDataAIServices.Deidentification.json#L214