We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SdkHttpResponse.contentTypes?
minor
SdkHttpResponse.contentTypes is an optional array:
SdkHttpResponse.contentTypes
typespec-azure/packages/typespec-client-generator-core/src/interfaces.ts
Lines 435 to 440 in d30db5b
So it would be better to either a non-empty array or undefined, but not an empty array.
undefined
Try to parse a spec like https://github.com/Azure/cadl-ranch/blob/main/packages/cadl-ranch-specs/http/authentication/http/custom/main.tsp
You'll find the contentTypes is an empty array.
contentTypes
The text was updated successfully, but these errors were encountered:
getAllOperation
good point, let me fix this
Sorry, something went wrong.
.contentTypes
[tcgc] don't set optional .contentTypes property on responses to an…
932666f
… empty list (#1070) fixes #992 --------- Co-authored-by: iscai-msft <[email protected]>
8e6e6e6
… empty list (Azure#1070) fixes Azure#992 --------- Co-authored-by: iscai-msft <[email protected]>
iscai-msft
Successfully merging a pull request may close this issue.
Describe the bug
minor
SdkHttpResponse.contentTypes
is an optional array:typespec-azure/packages/typespec-client-generator-core/src/interfaces.ts
Lines 435 to 440 in d30db5b
So it would be better to either a non-empty array or
undefined
, but not an empty array.Reproduction
Try to parse a spec like https://github.com/Azure/cadl-ranch/blob/main/packages/cadl-ranch-specs/http/authentication/http/custom/main.tsp
You'll find the
contentTypes
is an empty array.Checklist
The text was updated successfully, but these errors were encountered: