Closed keitakn closed 1 year ago
https://github.com/commew/timelogger-web/issues/4
各ソースコードをESLint, Prettierの警告、エラーに従い修正するのは別PRで対応します。
UI変更はないのでなし。
ESLint, Prettierの設定を現時点でベストと思われる内容に変更しました。
最終的に追加した関連packageの内容は以下の通りです。
npm install -D prettier @typescript-eslint/eslint-plugin @types/prettier @types/eslint eslint eslint-config-prettier eslint-config-standard-with-typescript eslint-plugin-import eslint-plugin-jsx-a11y eslint-plugin-n eslint-plugin-promise eslint-plugin-react eslint-plugin-react-hooks eslint-config-next
設定内容ですが、有名なReactの入門書籍である下記の書籍から拝借しています。(下記の書籍はNext.jsではないので、元々追加されていた eslint-config-next の設定は引き継いでいます。)
eslint-config-next
https://github.com/klemiwary/Riakuto-StartingReact-ja4.0
また npm-run-all を追加して複数のLinter, Formatterが実行出来る形に改修しました。
npm-run-all
ESLint, Prettierの設定に関して意見があれば教えて下さい!
情報共有の為、レビュアーに設定させて頂いております。
しばらくは開発が出来る状態までプロジェクトを整備している状況です。
お時間がありましたら、目を通して頂く程度の温度感で問題ありません。
※ 初期構築が完了した時点でフロントエンドメンバーには別途説明会の機会を作らせて頂きます。
特になし
issueURL
https://github.com/commew/timelogger-web/issues/4
このPRで対応する範囲 / このPRで対応しない範囲
本PRの対応範囲
各ソースコードをESLint, Prettierの警告、エラーに従い修正するのは別PRで対応します。
StorybookのURL、 スクリーンショット
UI変更はないのでなし。
変更点概要
ESLint, Prettierの設定を現時点でベストと思われる内容に変更しました。
最終的に追加した関連packageの内容は以下の通りです。
設定内容ですが、有名なReactの入門書籍である下記の書籍から拝借しています。(下記の書籍はNext.jsではないので、元々追加されていた
eslint-config-next
の設定は引き継いでいます。)https://github.com/klemiwary/Riakuto-StartingReact-ja4.0
また
npm-run-all
を追加して複数のLinter, Formatterが実行出来る形に改修しました。レビュアーに重点的にチェックして欲しい点
ESLint, Prettierの設定に関して意見があれば教えて下さい!
レビューについて
情報共有の為、レビュアーに設定させて頂いております。
しばらくは開発が出来る状態までプロジェクトを整備している状況です。
お時間がありましたら、目を通して頂く程度の温度感で問題ありません。
※ 初期構築が完了した時点でフロントエンドメンバーには別途説明会の機会を作らせて頂きます。
補足情報
特になし