Skip to content

Conversation

kimkim0814
Copy link
Member

@kimkim0814 kimkim0814 commented Aug 13, 2024

What

  • Qiita CLIのNode.jsのサポートバージョンを>=18.18.0にする
    • パッケージのアップデートの際にNodeバージョンが古くアップデートに支障が出たため
  • 開発環境のNode.jsのバージョンを20.16.0にする

How

  • Node18.12.0 > 18.18.0 にした
  • READMEに記載していた最低バージョンの表記を修正した
  • CI時のテストのバージョンを変更した

Why

Refs

@kimkim0814 kimkim0814 self-assigned this Aug 13, 2024
@kimkim0814 kimkim0814 force-pushed the update-from-node-18.12.0-to-18.18.0 branch from ba54e7b to 63a6252 Compare August 13, 2024 06:18
@kimkim0814 kimkim0814 closed this Aug 13, 2024
@kimkim0814 kimkim0814 reopened this Aug 13, 2024
@kimkim0814
Copy link
Member Author

kimkim0814 commented Aug 13, 2024

📝
変更前のCIが実行状態になっている。
キャッシュの問題ではなさそう(?)

スクリーンショット 2024-08-13 15 52 35 スクリーンショット 2024-08-13 15 59 34

ref: https://github.com/increments/qiita-cli/actions/caches


上記を解決するために行ったこと

  • PRをcloseしてreopen
    • 解決せず

仮説

@kimkim0814
Copy link
Member Author

kimkim0814 commented Aug 13, 2024

次のアクション

  • protection rulesから不適切なルールを外し、新たにルールを追加する
    • 外す: test (18.12.0) test (18.19.0) test (20.11.0)
    • 追加: test (18.18.0) test (18.20.0) test (20.16.0)
  • CIが問題なくパスするかどうかを確認する
    • パスしない場合は、適宜修正を行う
  • Change logを確認して、Qiita Cli内で修正箇所があるか確認する
    • File Classの導入(18.13.0)
    • npmが 9.3.1に更新(18.14.0)
    • 特定の文字を含むファイル名を扱えないなど、Bug Fixが主(18.18.0)
  • 開発環境で動作確認を行う
  • アップデート後に、nodeのサポートバージョンの関係でアップデートできなかったライブラリをアップデートする
@ohakutsu ohakutsu force-pushed the update-from-node-18.12.0-to-18.18.0 branch from 63a6252 to b52748a Compare August 14, 2024 12:49
@ohakutsu ohakutsu changed the title Update from node 18.12.0 to 18.18.0 Drop support for Node.js <18.18.0 Aug 14, 2024
@ohakutsu ohakutsu requested review from a team and wataru86 and removed request for a team August 14, 2024 12:57
@ohakutsu ohakutsu force-pushed the update-from-node-18.12.0-to-18.18.0 branch 2 times, most recently from dba7577 to 42f7401 Compare August 14, 2024 13:01
Copy link
Member

@wataru86 wataru86 left a comment

Choose a reason for hiding this comment

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

LGTM!

@ohakutsu ohakutsu self-assigned this Aug 15, 2024
@ohakutsu ohakutsu force-pushed the update-from-node-18.12.0-to-18.18.0 branch from 42f7401 to 831449d Compare August 15, 2024 08:49
@ohakutsu ohakutsu merged commit c783e6f into main Aug 16, 2024
@ohakutsu ohakutsu deleted the update-from-node-18.12.0-to-18.18.0 branch August 16, 2024 03:47
@ohakutsu ohakutsu mentioned this pull request Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants