Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

npm packageを最新安定版に更新 #267

Merged
merged 4 commits into from
Dec 15, 2023

Conversation

keitakn
Copy link
Member

@keitakn keitakn commented Dec 15, 2023

issueURL

#262

関連 URL

https://lgtm-cat-frontend-git-feature-issue262upgrade-7b6f99-nekochans.vercel.app/

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

npm packageが可能な限り最新安定版に更新する。

mswに関しては #264 で対応するのでここでは対応しない。

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

https://622b6c5dc31e9e003a111eb5-atogsqdnzo.chromatic.com/?path=/story/components-errorcatimages-internalservererrorimage--default

変更点概要

Next.jsを含む依存packageを最新安定版に更新。

stylelint に関しては16系が最新だが、それだとプラグインが対応していないようなので15系の最新にアップグレードした。

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

特になし

補足情報

インラインコメントに記載

@keitakn keitakn self-assigned this Dec 15, 2023
Copy link

vercel bot commented Dec 15, 2023

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 Dec 15, 2023 2:34pm

Copy link

codecov bot commented Dec 15, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (961aba0) 74.94% compared to head (03dc935) 74.94%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #267   +/-   ##
=======================================
  Coverage   74.94%   74.94%           
=======================================
  Files          47       47           
  Lines         443      443           
  Branches      109      109           
=======================================
  Hits          332      332           
  Misses        107      107           
  Partials        4        4           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -6,7 +6,7 @@

[optional]

# Done の定義
# このPRで対応すること / このPRで対応しないこと
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PRのテンプレートを修正。こちらのほうが何を書くか明確なので。

@keitakn keitakn marked this pull request as ready for review December 15, 2023 14:53
@keitakn keitakn merged commit 27ce77c into main Dec 15, 2023
5 checks passed
@keitakn keitakn deleted the feature/issue262/upgrade-npm-packages branch December 15, 2023 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant