-
-
Notifications
You must be signed in to change notification settings - Fork 962
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
nodejs-lts: fix checkver and add arm64 version #5657
Conversation
All changes look good. Wait for review from human collaborators. nodejs-lts
|
/verify |
All changes look good. Wait for review from human collaborators. nodejs-lts
|
For what it's worth: I'm not sure how to review this, but I tested it by running this: scoop install https://raw.githubusercontent.com/ScoopInstaller/Main/0f5202c216bf79be81549880c86f33eef1218be9/bucket/nodejs-lts.json (that's the raw URL of the file in this PR) and it worked flawlessly on my computer. Thanks! |
Seeing how long it takes to for those PRs to get merged, we should change the checkver script to instead get the latest version from https://nodejs.org/dist/index.json to no longer be dependent on the layout of nodejs.org. |
/verify |
All changes look good. Wait for review from human collaborators. nodejs-lts
nodejs
|
Due to the redesign of the Node.js website, the checkver of the nodejs-lts package is broken, this pull request fixes it, and also adds the arm64 version