search

macmeals / todo_context_typescript

TodoリストのTypeScript化
0 stars 0 forks source link

型定義の命名規則について #6

Closed mizushima1226 closed 2 years ago

mizushima1226 commented 2 years ago

https://typescript-jp.gitbook.io/deep-dive/styleguide#%E3%82%BF%E3%82%A4%E3%83%97

type や interfaceはパスカルケースを使うので修正しておいた方が良いかと思います〜

eslintの設定で保証できるのでこちらおすすめです〜 https://dev.classmethod.jp/articles/shuntaka9576-check-eslint/

mizushima1226 commented 2 years ago

また、コンポーネント内に型定義している箇所いくつか見受けれますが、型定義はReactのステートに依存しないので コンポーネントの外側に書くのが一般的です👍

macmeals commented 2 years ago

お疲れ様です！こちらのIssueに対してのPullRequestを上げました。確認いただけますでしょうか？

macmeals commented 2 years ago

お疲れ様です。先ほどPullリクエストを修正いたしました。ご確認いただけますでしょうか？

macmeals commented 2 years ago

ご確認ありがとうございました！先ほどプルリクエストをマージしたので、こちらクローズします〜