-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changing schema from string to file #3420
Conversation
Automation for azure-sdk-for-javaNothing to generate for azure-sdk-for-java |
Automation for azure-sdk-for-pythonThe initial PR has been merged into your service PR: |
Automation for azure-sdk-for-rubyThe initial PR has been merged into your service PR: |
Automation for azure-sdk-for-nodeThe initial PR has been merged into your service PR: |
Automation for azure-sdk-for-goNothing to generate for azure-sdk-for-go |
Thanks @vrdmr! Additionally I see in the previous PR that @hovsepm already let you know about the breaking changes to the SDKs - do you know if any packages were published between the previous PR and this one? |
Hi @annatisch, Validation errors were not removed as I don't know of a clear way to describe a body as file and not a string - especially for this issue: And to answer your second question - No packages were published. This issue was caught during the tests for releasing a package. |
@AutorestCI regenerate azure-sdk-for-go |
@jhendrixMSFT For Go, do you need something to be added to our readme.md for solving the |
@vrdmr No it looks like this package is already current in the Go SDK which is why there was nothing to generate. |
Forgot to add this bit in the #3363. Reverting the change of body parameter schema to be of type:object/format file, as it was earlier.
@hovsepm Could you please take a look.
PR information
api-version
in the path should match theapi-version
in the spec).Quality of Swagger