nekochans / lgtm-cat-frontend

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

App Routerへ移行前にNext.jsと関連packageを最新安定版に更新 #282

Closed keitakn closed 6 months ago

keitakn commented 6 months ago

issueURL

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

関連 URL

https://lgtm-cat-frontend-git-feature-issue280upgrade-9a8010-nekochans.vercel.app/

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

Next.jsと関連packageを最新安定版に更新する。

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

変更点概要

App Routerへ移行前にNext.jsと関連packageを最新安定版に更新。

tsconfig.jsonnext.config を最新のテンプレートに合わせて修正。

ただし tsconfig.json は最新のテンプレートだと moduleResolutionbundler になっているがこれだと @nekochans/lgtm-cat-ui がimportできないので今は node にしている。

@nekochans/lgtm-cat-ui のComponentは全て lgtm-cat-frontend 側に移行するのでその時に bundler に変更する。

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

特になし

補足情報

特になし

vercel[bot] commented 6 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 Feb 12, 2024 6:02am
codecov[bot] commented 6 months ago

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (6c8acde) 60.35% compared to head (92dc378) 59.66%.

Files Patch % Lines
next.config.mjs 0.00% 4 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #282 +/- ## ========================================== - Coverage 60.35% 59.66% -0.69% ========================================== Files 92 93 +1 Lines 3032 3067 +35 Branches 155 156 +1 ========================================== Hits 1830 1830 - Misses 1157 1191 +34 - Partials 45 46 +1 ```

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