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

alternative version restriction #2

Closed
wants to merge 1 commit into from
Closed

alternative version restriction #2

wants to merge 1 commit into from

Conversation

marihachi
Copy link

Solution for #1

@marihachi marihachi requested a review from syuilo May 30, 2019 15:53
@u1-liquid
Copy link
Collaborator

u1-liquid commented May 30, 2019

私の知る限り.node-versionってnvm https://github.com/nvm-sh/nvm ないしnodenv https://github.com/nodenv/nodenv の為のもので、Node.jsのバージョンを強制する為のファイルではないんですよね

@syuilo
Copy link
Owner

syuilo commented May 31, 2019

node-versionはnvmとかのために設置したのでそのままで良さそう
(そこまで厳密にバージョン指定する意図はない)

@marihachi
Copy link
Author

nodistも.node-versionを見ていて、微妙にフォーマットが違うために実行に失敗する

@tamaina
Copy link

tamaina commented May 31, 2019

バージョンチェックを自分でやるならせめてsemverあたりを使って範囲指定可能にしてほしい

@syuilo
Copy link
Owner

syuilo commented May 31, 2019

一応起動時にバージョンチェックするようにしたけどどうでしょう

@marihachi
Copy link
Author

.node-versionがあるとNodeの起動ができなくなるので、削除したいんだけどどうかな

@u1-liquid
Copy link
Collaborator

u1-liquid commented May 31, 2019

ちょっと調べたらnodistの.node-version形式はx.y.z(先頭にvがない)ですがparserでvはtrimされるので特に問題なさそう
ただnodistにv12.1.0がインストールされていないのでは

ちなみにnodevx.y.z形式を期待するnodistは旧バージョンなのでアップグレードしてください

@marihachi
Copy link
Author

@u1-liquid 情報ありがとう
nodistのバージョン古そうなので最新版入れ直してみます!

@marihachi
Copy link
Author

Nodistのアップデートにて解決しました。お騒がせしました。。
協力してくださりありがとうございます。

@marihachi marihachi closed this Jun 1, 2019
@marihachi marihachi deleted the #1 branch June 1, 2019 02:54
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.

None yet

4 participants