Skip to content

Improve model class of ruby-client#1670

Merged
wing328 merged 4 commits intoOpenAPITools:masterfrom
meganemura:ruby-improve-build
Dec 15, 2018
Merged

Improve model class of ruby-client#1670
wing328 merged 4 commits intoOpenAPITools:masterfrom
meganemura:ruby-improve-build

Conversation

@meganemura
Copy link
Contributor

PR checklist

  • Read the contribution guidelines.
  • Ran the shell 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). Windows batch files can be found in .\bin\windows\.
  • Filed the PR against the correct branch: master, 3.4.x, 4.0.x. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

In ruby client, we use many Model.new.build_from_hash.
This change removes redundant .new.

@wing328
Copy link
Member

wing328 commented Dec 14, 2018

@meganemura the PR looks good to me. I'll merge it over the weekend if no one has further question/feedback.

Have a nice weekend!

cc @cliffano (2017/07) @zlx (2017/09)

@wing328 wing328 merged commit 82fcde1 into OpenAPITools:master Dec 15, 2018
@meganemura meganemura deleted the ruby-improve-build branch December 18, 2018 08:36
@meganemura
Copy link
Contributor Author

Thanks 😄

@wing328
Copy link
Member

wing328 commented Jan 2, 2019

@meganemura thanks for the contribution, which has been included in the 4.0.0-beta release: https://twitter.com/oas_generator/status/1079727020374806529.

Happy New Year and looking forward to more collaboration and contributions in 2019!

A-Joshi pushed a commit to ihsmarkitoss/openapi-generator that referenced this pull request Feb 27, 2019
* Add Model.build_from_hash

* Use Model.build_from_hash instead of Model.new.build_from_hash

* Update sample for ruby-client

* Update sample for openapi3 ruby-client
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

Comments