Skip to content

Conversation

@wing328
Copy link
Contributor

@wing328 wing328 commented Mar 21, 2017

PR checklist

  • Read the contribution guidelines.
  • 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

To fix #4539 (comment)

@wing328
Copy link
Contributor Author

wing328 commented Mar 21, 2017

Related PR: #5109

@kernwig
Copy link
Contributor

kernwig commented Mar 21, 2017

Looks good @wing328

@wing328 wing328 merged commit 1c32948 into swagger-api:master Mar 21, 2017
@wing328 wing328 deleted the ts_import_apifilename branch March 21, 2017 15:52
@wing328
Copy link
Contributor Author

wing328 commented Mar 21, 2017

I'll sync the change to 2.3.0 tomorrow

I also open #5136 to fix similar issues, if any, in other TS generators.

spr3nk3ls pushed a commit to spr3nk3ls/swagger-codegen that referenced this pull request Mar 28, 2017
…5131)

* fix class filename in ts angular2 template

* remove trailing comma in ts angular2 template
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