search

nekochans / lgtm-cat-frontend

lgtm-cat（サービス名 LGTMeow https://lgtmeow.com) のフロントエンド用
https://lgtmeow.com
22 stars 2 forks source link

Next.js + Tailwind CSSで開発出来るようにProject構成を変更 #340

Closed keitakn closed 2 months ago

keitakn commented 2 months ago

issueURL

https://github.com/nekochans/lgtm-cat-frontend/issues/337

関連 URL

https://lgtm-cat-frontend-git-feature-issue337reset-project-nekochans.vercel.app/

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

Tailwind CSS + Next.jsで開発が出来るようにProject構成を変更する。

Storybook の URL もしくはスクリーンショット

https://622b6c5dc31e9e003a111eb5-kejprogtks.chromatic.com/

変更点概要

タイトルの通りNext.js + Tailwind CSSで開発出来るようにProject構成を変更 している。

一旦差分が分かりやすいように新しいProjectに必要なファイルだけ残しているが、残りの機能の関しても必要な物は順次 main ブランチから移行していく。

以前までの構造だと各ディレクトリに index.ts を置くスタイル（バレルファイルというらしい）を採用していたが今回からバレルファイルは使わない構成としている。

理由は下記の通りで以外とBuild時のパフォーマンスに影響を与えるようだ。

https://marvinh.dev/blog/speeding-up-javascript-ecosystem-part-7/

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

特になし

補足情報

特になし

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lgtm-cat-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 17, 2024 5:46pm
codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 3.03030% with 128 lines in your changes missing coverage. Please review.

Project coverage is 14.86%. Comparing base (8fc0b2c) to head (c8978bc). Report is 16 commits behind head on release/epic36/design-renewal-2024.

Files Patch % Lines
src/app/page.tsx 0.00% 34 Missing and 1 partial :warning:
src/app/_components/Button.stories.tsx 0.00% 26 Missing and 1 partial :warning:
src/app/_components/Button.tsx 0.00% 21 Missing and 1 partial :warning:
src/app/layout.tsx 0.00% 11 Missing :warning:
postcss.config.mjs 0.00% 7 Missing and 1 partial :warning:
src/middleware.ts 0.00% 5 Missing and 1 partial :warning:
eslint.config.mjs 0.00% 2 Missing :warning:
.../mocks/api/external/lgtmeow/mockFetchLgtmImages.ts 0.00% 1 Missing and 1 partial :warning:
src/app/global-error.tsx 0.00% 1 Missing :warning:
src/edge/url.ts 0.00% 0 Missing and 1 partial :warning:
... and 13 more

:exclamation: There is a different number of reports uploaded between BASE (8fc0b2c) and HEAD (c8978bc). Click for more details.

HEAD has 1 upload less than BASE | Flag | BASE (8fc0b2c) | HEAD (c8978bc) | |------|------|------| ||2|1|
Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/epic36/design-renewal-2024 #340 +/- ## ======================================================================= - Coverage 30.49% 14.86% -15.63% ======================================================================= Files 262 47 -215 Lines 6746 1164 -5582 Branches 354 71 -283 ======================================================================= - Hits 2057 173 -1884 + Misses 4535 952 -3583 + Partials 154 39 -115 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.