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

Voltaによるバージョン管理を追加 #1306

Open
3 tasks
My-MC opened this issue May 3, 2023 · 7 comments
Open
3 tasks

Voltaによるバージョン管理を追加 #1306

My-MC opened this issue May 3, 2023 · 7 comments

Comments

@My-MC
Copy link
Contributor

My-MC commented May 3, 2023

内容

Voltaによるバージョンの管理をします。
具体的にはpackage.json内にvoltaのnode.jsとnpmのバージョンを書きます。

Pros 良くなる点

Voltaには自動バージョン切り替え機能があり開発者が利用しやすくなる。
利用者の多いVoltaをサポートすることでよりプロジェクトに参加しやすくなる。

Cons 悪くなる点

.node_versionと一緒に管理しなくてはいけなくなる。

実現方法

volta pinコマンドで追加する

VOICEVOXのバージョン

0.?.0

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

@Hiroshiba
Copy link
Member

Hiroshiba commented May 4, 2023

Voltaを調べていました。.node-versionを参照しないんですね。
OSSプロジェクトでVoltaの設定をpackage.jsonに追加しているものを調べてみたのですが、lodashとかvuejsとかを見る感じはついていませんでした。

OSSに関係ないのでなるべく追加したくない思いがあります。
といっても.node-versionファイルはあるので、こちらと同期させる仕組みがある(乖離しているとエラーが出る)のであれば導入してもいいかなと思いました。

(個人的には、.node-versionに対応していないのはgitignoreできないとうことなので、VoltaはOSSとの親和性がちょっと低いんだなーと思いました。)

@y-chan
Copy link
Member

y-chan commented May 4, 2023

Voltaを導入して使ったことがありますが、基本的には少人数開発や会社等の限られた人数で開発されるもので使うかなといった感じでした。
Voltaを使えば確実にNode.jsのバージョンを合わせられるので良いとは思いますが、OSSに対して適用するのは少し慎重になった方がいいというのが個人的な意見になります。
また、このリポジトリにおいてyarnの導入が見送られた( #70 )のと同じ理由で、初学者や、新規コントリビュータが参画しづらくなる障壁になりうるのではないかと考えています。

@My-MC
Copy link
Contributor Author

My-MC commented May 5, 2023

.node_versionとVoltaが同期しているかを確認するGitHub Actionを作ってみました。My-MC/check-sync-volta-and-node-version

機能はほぼないですが、その代わりだいたいかかって5秒くらいで確認できます。

@Hiroshiba
Copy link
Member

@y-chan なるほどです!
Voltaを必須にするのは避けた方が良さそうだなと思いました。
オプショナルとして選択できるようにしておくのは、メンテしてくださる方がいるならまぁアリかな〜という気持ちです。

@My-MC 検査Actionがあるならpackage.jsonに追記もありだと思います!
普通のnpmも使える、という理解であってますよね…?

@My-MC
Copy link
Contributor Author

My-MC commented May 6, 2023

@My-MC 検査Actionがあるならpackage.jsonに追記もありだと思います!
普通のnpmも使える、という理解であってますよね…?

はいあってます。リポジトリの方でyarnを指定していますが、npmも指定できます。

@Hiroshiba
Copy link
Member

voltaを使わずとも普通にnpmも使えて、バージョンの整合性チェックができてるなら追記しても良いのかなと思いました!

@My-MC
Copy link
Contributor Author

My-MC commented May 9, 2023

ではPR作ります

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants