-
Notifications
You must be signed in to change notification settings - Fork 196
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
エンジン情報に関するmodelを追加 #378
エンジン情報に関するmodelを追加 #378
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
良さそうです!!
細かいのですが、語感をどうすれば良いか難しそうだったのでコメントしてみました!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
コメントしました!
指摘部分の修正を行いました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
Manifestすごく良いなと思いました!!
細かい数点だけちょっとsuggestしてみたので、問題なさそうであれば含めて頂けると・・・!
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: aoirint <aoirint@gmail.com>
マージしちゃいます! |
内容
題の通りです。
実際に情報を返す部分まで実装するとPRが大きくなるため、modelのみのPRとしています。
もっと良さそうな命名や追加した方が良い情報があればコメント等お願いします。
GET /engine_info
、GET /supported_functions_info
で情報を取得できるようにしようと考えています。関連 Issue