Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

APIクライアントを追加 #1

Merged
merged 7 commits into from
Jun 28, 2018
Merged

Conversation

ackintosh
Copy link
Contributor

@ackintosh ackintosh commented Jun 21, 2018

https://github.com/OpenAPITools/openapi-generator で生成したAPIクライアントを追加します 👷


📝

  • generate.sh でコードを生成
$ ./generate.sh
[main] INFO org.openapitools.codegen.DefaultCodegen - Skipped overwriting .rspec as the file already exists in /local//.rspec
[main] INFO org.openapitools.codegen.DefaultCodegen - Skipped overwriting spec_helper.rb as the file already exists in /local/spec/spec_helper.rb
[main] INFO org.openapitools.codegen.DefaultCodegen - Skipped overwriting configuration_spec.rb as the file already exists in /local/spec/configuration_spec.rb
[main] INFO org.openapitools.codegen.DefaultCodegen - Skipped overwriting api_client_spec.rb as the file already exists in /local/spec/api_client_spec.rb
[main] INFO org.openapitools.codegen.AbstractGenerator - writing file /local/lib/color_me_shop/models/customer.rb
[main] INFO org.openapitools.codegen.AbstractGenerator - writing file /local/docs/Customer.md
[main] INFO org.openapitools.codegen.AbstractGenerator - writing file /local/lib/color_me_shop/models/delivery.rb
[main] INFO org.openapitools.codegen.AbstractGenerator - writing file /local/docs/Delivery.md
...
...
...
  • テストコードは要らないのでは?と思ってgenerate.shで -DmodelTests=false -DapiTests=false を指定している
    • このリポジトリには自動生成したコードしかないので 💡
  • generate.sh で生成する 旨をどこに書いておこう 🤔
    • README.mdは自動生成してるので出来ればいじりたくない
    • リポジトリの説明欄??
      • 現在: カラーミーショップAPIのRubyクライアントです 💎
      • 変更後: カラーミーショップAPIのRubyクライアントです 💎 generate.sh で生成しています

@ackintosh
Copy link
Contributor Author

@ackintosh
Copy link
Contributor Author

ackintosh commented Jun 24, 2018

descriptionが見辛い件、issueをたてた 📝
OpenAPITools/openapi-generator#369


(追記)
よく考えたらカラーミーAPIのdescriptionの項目では、APIの使い方( はじめに の内容)を書いてるので、そもそもRubyクライアントのREADMEには不要なのではという気持ちになってきた 💭 けど上書きする手段は無い 👁

@ackintosh ackintosh changed the title [WIP] APIクライアントを追加 APIクライアントを追加 Jun 27, 2018
#!/bin/bash

docker run --rm -v ${PWD}:/local openapitools/openapi-generator-cli generate \
-DmodelTests=false -DapiTests=false \
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ackintosh
Copy link
Contributor Author

見辛い件はおいておいて、レビューをお願いしよう 🙏

@yinm
Copy link
Contributor

yinm commented Jun 28, 2018

コードはよさそうです!

ただ、1点確認です。

変更後: カラーミーショップAPIのRubyクライアントです 💎 generate.sh で生成しています

これってもうやってます??
https://github.com/pepabo/colormeshop-ruby の Descriptionかなと思ったけど、違うっぽかったので〜!

@ackintosh
Copy link
Contributor Author

レビュー :azs: !

まだやってないです!
"カラーミーショップAPIのRubyクライアントです 💎 generate.sh で生成しています" で伝わるかな〜というのもレビューしていただきたいのでした 🙏 💦

Copy link

@usasan usasan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

良さそうです。

@ackintosh
Copy link
Contributor Author

ありがとうございます〜 ₍₍ (ง ˘ω˘ )ว ⁾⁾

@ackintosh ackintosh merged commit b38fdb4 into pepabo:master Jun 28, 2018
@ackintosh ackintosh deleted the generate-client branch June 28, 2018 07:47
@yinm
Copy link
Contributor

yinm commented Jun 28, 2018

"カラーミーショップAPIのRubyクライアントです 💎 generate.sh で生成しています" で伝わるかな〜というのもレビューしていただきたいのでした 🙏 💦

なるほどです!
こちらもよさそうです!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants