commew / timelogger-web

時間記録アプリ
https://timmew.commew.net
MIT License
2 stars 0 forks source link

プロジェクトの初期構成を追加 #14

Closed keitakn closed 1 year ago

keitakn commented 1 year ago

issueURL

https://github.com/commew/timelogger-web/issues/2

このPRで対応する範囲 / このPRで対応しない範囲

ESLintの細かい設定などはここでは対応しない。あくまでも機械的に生成されたコードを追加するだけ。

StorybookのURL、 スクリーンショット

スクリーンショット 2023-01-27 16 20 52

変更点概要

npx create-next-app@latest が生成したファイルをコミットした。

create-next-app コマンドの公式ドキュメントは下記の通り。

https://nextjs.org/docs/api-reference/create-next-app

こちらのコマンドは対話式になっており、それぞれ以下のように回答した。

Need to install the following packages:
  create-next-app@13.1.5
Ok to proceed? (y) y
✔ What is your project named? … next-example
✔ Would you like to use TypeScript with this project? … No / Yes
✔ Would you like to use ESLint with this project? … No / Yes
✔ Would you like to use `src/` directory with this project? … No / Yes
✔ Would you like to use experimental `app/` directory with this project? … No / Yes
✔ What import alias would you like configured? … @/*
Creating a new Next.js app in /Users/kogakeita/gitrepos/next-example/next-example.

まとめると以下の通り。

Next.js 13から新しく出た app directory は無効になっています。

有効にしたほうがコードの設計面や性能面で優位にはなりますが、app directoryはベータ版の機能なので安定性を重視し今回は採用しない方針としました。(安定版になった後で乗り換える事は可能です)

(参考)https://zenn.dev/azukiazusa/articles/next-js-app-dir-tutorial

レビュアーに重点的にチェックして欲しい点

情報共有の為、レビュアーに設定させて頂いております。

しばらくは開発が出来る状態までプロジェクトを整備している状況です。

お時間がありましたら、目を通して頂く程度の温度感で問題ありません。

※ 初期構築が完了した時点でフロントエンドメンバーには別途説明会の機会を作らせて頂きます。

補足情報

PRの変更点が6000行を超えており本来であればこのような大きなPRは望ましくないですが、自動生成されたコードがほとんどなので、PRを分けるとかえって分かりにくいのでこのまま出しています。

そのほか、インラインコメントに記載してある点をご確認下さい。