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

Required Node.js LTS #97

Closed
akabekobeko opened this issue May 9, 2021 · 3 comments
Closed

Required Node.js LTS #97

akabekobeko opened this issue May 9, 2021 · 3 comments
Assignees
Milestone

Comments

@akabekobeko
Copy link
Member

In Node.js LTS, 12 is maintained and 14 is active. Therefore, the lower limit of support is set to 12.

@akabekobeko akabekobeko added this to the v1.0.0 milestone May 9, 2021
@akabekobeko akabekobeko self-assigned this May 9, 2021
@akabekobeko
Copy link
Member Author

@MurakamiShinyu @spring-raining
現在の VFM は package.jsonengine が未指定であるため、サポートする Node.js バージョンを定めていません。しかし依存 npm には下限を設定しているものが幾つかあり、GitHub の Security Alert を直すために更新の必要な meow などは 12 を下限としています。

現時点の Node.js LTS は 12 が MAINTENANCE、14 を ACTIVE としており、VFM が依存する npm でも 12 下限があるのでこれを定義しようと考えています。ただし vivliostyle-cli は "node": ">=10 <=12 || >=14.1.0" なので、VFM を 12 にすると 10 環境でインストールに失敗すると思われます。

以上を踏まえて相談なのですが VFM としてはどのように対応するのがよいでしょうか?

  1. "node": ">= 12" を設定
    • VFM の依存 npm を更新できる
    • vivliostyle-cli もこれを下限とする必要あり
  2. "node": ">= 10" を設定
    • VFM の依存 npm を更新できない
    • vivliostyle-cli の対応は不要
  3. その他

関連する提案として Vivliostyle 関連プロジェクトの Node.js サポート方針について取り決めたいです。create-book や vivliostyle-cli のように別の関連プロジェクトへ依存する場合、統一された方針が必要だと考えています。Node.js LTS の MAINTENANCE または ACTIVE を下限とするのがよさそうです。

@akabekobeko akabekobeko changed the title Node.js 12 Required Node.js 12 May 9, 2021
@akabekobeko akabekobeko changed the title Required Node.js 12 Required Node.js LTS May 9, 2021
@akabekobeko
Copy link
Member Author

akabekobeko commented Jul 20, 2021

2021/7 の開発者会議 にて @spring-raining さんとしては

CLIとしてはRequirement versionをv12に上げるのは問題ないです

とのこと。

に VFM は

{
  "engines": {
      "node": ">= 12"
  }
}

とする旨をコメントしたうえで移行する予定。

@akabekobeko
Copy link
Member Author

本件は #118 にて対応された。v1.0.0 に反映予定。close する。

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

No branches or pull requests

1 participant