updating spec for file type to comply with schema

[veronicagg]

Face.json is currently not complying with swagger 2.0 spec (per schema), which is currently breaking CI in the repo.
Swagger 2.0 Schema: https://github.com/OAI/OpenAPI-Specification/blob/master/schemas/v2.0/schema.json
which is the base for the extended one we use at https://github.com/Azure/autorest/blob/master/schema/swagger-extensions.json

See related thread: OAI/OpenAPI-Specification#1226

Making an update to an equivalent type/format, which produces the same generated code via AutoRest.

[vishrutshah]

Changes LGTM! I'll re-start the CI once PR Azure/autorest#2529 is merged for CI to pass.

[azuresdkciprbot]

Hi There,

I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:

File: specification/cognitiveservices/data-plane/Face/readme.md
Before the PR: Warning(s): 87 Error(s): 16
After the PR: Warning(s): 87 Error(s): 16

AutoRest Linter Guidelines | AutoRest Linter Issues

Send feedback and make AutoRest Linter Azure Bot smarter day by day!

Thanks for your co-operation.

[fearthecowboy]

o_O

Why are we reverting this and switching the way that it's specified?

I thought we were good with

"schema": {
  "type":"file" 
}

given that the spec (https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md#data-types) clearly states:

Primitive data types in the Swagger Specification are based on the types supported by the JSON-Schema Draft 4. Models are described using the Schema Object which is a subset of JSON Schema Draft 4.
An additional primitive data type "file" is used by the Parameter Object and the Response Object to set the parameter type or the response as being a file.

[veronicagg]

talked to @fearthecowboy offline and agreed on making this change at this time, to comply with swagger 2.0 schema. See links referenced above for more info.

[azuresdkciprbot]

Hi There,

I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:

File: specification/cognitiveservices/data-plane/Face/readme.md
Before the PR: Warning(s): 87 Error(s): 16
After the PR: Warning(s): 87 Error(s): 16

AutoRest Linter Guidelines | AutoRest Linter Issues

Send feedback and make AutoRest Linter Azure Bot smarter day by day!

Thanks for your co-operation.

[AutorestCI]

No modification for AutorestCI/azure-sdk-for-node

[AutorestCI]

No modification for AutorestCI/azure-sdk-for-ruby

[AutorestCI]

No modification for AutorestCI/azure-sdk-for-python