Skip to content

[Swift] Fix issue #4764#4787

Closed
AnthonyOliveri wants to merge 1 commit intoswagger-api:masterfrom
ibm-bluemix-mobile-services:master
Closed

[Swift] Fix issue #4764#4787
AnthonyOliveri wants to merge 1 commit intoswagger-api:masterfrom
ibm-bluemix-mobile-services:master

Conversation

@AnthonyOliveri
Copy link
Copy Markdown
Contributor

@AnthonyOliveri AnthonyOliveri commented Feb 13, 2017

PR checklist

  • Read the contribution guildelines.
  • Ran the shell/batch script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)
  • Filed the PR against the correct branch: master for non-breaking changes and 2.3.0 branch for breaking (non-backward compatible) changes.

Description of the PR

Adds file type detection to Swift codegen, which was previously missing. This was causing Swift codegen to never know when a parameter or response type was defined with type file.

This fixes issue #4764.

@AnthonyOliveri
Copy link
Copy Markdown
Contributor Author

I moved the fork to a different repository, so I am replacing this pull request with #5065.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants