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

nodejsを16に更新 #890

Merged
merged 6 commits into from
Sep 10, 2022
Merged

nodejsを16に更新 #890

merged 6 commits into from
Sep 10, 2022

Conversation

sousuke0422
Copy link
Contributor

@sousuke0422 sousuke0422 commented Aug 8, 2022

内容

nodejsを16に更新します。
npmも更新が入っています。

関連 Issue

ref #850
resolve #464

スクリーンショット・動画など

その他

lockfileVersionの混乱に終止符を打ちます

@Hiroshiba
Copy link
Member

PRありがとうございます!!

package-lockが

27,586 additions, 363 deletions

となっていて、deleteに比べ追加が非常に多そうでした。
何が増えてる感じなんでしょうか 👀

@sousuke0422
Copy link
Contributor Author

sousuke0422 commented Aug 8, 2022

npm iを実行してlockfileVersion: 2に更新するとlock.json内にpackagesが生成されます。さらにdevDependenciesかを識別する要素が追加されて要るっぽいです。

@sousuke0422
Copy link
Contributor Author

packagesは6~26269行あたりまでですね

@Hiroshiba
Copy link
Member

なるほどです・・・!!

@sousuke0422 sousuke0422 marked this pull request as ready for review August 9, 2022 04:15
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!!

いろいろ動かしてみたのですが、動作的に変わっている部分はなさそうに感じました!

1つちょっとお願いがあって、レビュー期間が空いてしまった間に、mainブランチでのquaserのバージョンが2.4.13にバージョンダウンしました。
なので、こちらのPRのブランチにmainをマージした後、npm iしてlock.jsonを更新いただいてもよろしいでしょうか。
お忙しければ、いったんマージしたあと即座に僕からPRを出そうかなと思います。

どちらでも大丈夫です!

@sousuke0422
Copy link
Contributor Author

npm iのついでにnodeもバージョン上げておきました

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!

マージします・・・!

@Hiroshiba Hiroshiba merged commit ab72c91 into VOICEVOX:main Sep 10, 2022
@Hiroshiba
Copy link
Member

こちらにてアナウンスをさせて頂きました 🙏

@sousuke0422 sousuke0422 deleted the feat/update-node branch September 12, 2022 04:00
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.

開発に使用するNode.jsのバージョンを上げたい
2 participants