Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
azu committed Apr 29, 2023
1 parent 5df97f7 commit 239e233
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions _posts/2023/2023-04-29-node.js-14-to-18.md
Original file line number Diff line number Diff line change
Expand Up @@ -154,10 +154,10 @@ Node.jsのアップデートする際に、Node.jsとライブラリを同時に
実際にNode.js 18へのアップデートをいくつかやってみて、Corepackを使った方法が実際の差分やPRの小さく作って進められたのでよかったと思います(`--legacy-peer-deps`も多分同じステップを取れますが、好みではなかった)。
実際の作業も、次のようなステップに分けて進めることができ、実際に出したPull Requestもそれぞれのステップごとにできました。

1. CI: Ubuntuマシンイメージのアップデート
2. (Node.js 14のまま): プロジェクトのインストールステップやDockerfileでcorepackを有効化
3. (Node.js 14のまま): Node.js 18では動かなかったライブラリをアップデート
1. (Node.js 14のまま): プロジェクトのインストールステップやDockerfileでcorepackを有効化
2. (Node.js 14のまま): Node.js 18では動かなかったライブラリをアップデート
- 具体的にはJestがReferenceError: AbortSignal is not definedで壊れていました
3. CI: Ubuntuマシンイメージのアップデート
4. Node.js のバージョン番号だけを変更する
- Dockerfileの`FROM node:14.x.x``FROM node:18.16.0`に変更
- CIのバージョン指定を変更
Expand Down

0 comments on commit 239e233

Please sign in to comment.