-
Notifications
You must be signed in to change notification settings - Fork 31.2k
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
doc: add riscv64 to list of architectures #49284
Conversation
If anyone is wondering where to get such a riscv64 Node.js build, see https://unofficial-builds.nodejs.org/ (nodejs/unofficial-builds#54). |
PR-URL: nodejs#49284 Signed-off-by: Stewart X Addison <sxa@redhat.com>
(Force push to correct the PR_URL as I'd cut & pasted the issue link instead of the PR) |
Setting the PR URL yourself in the commit message is really not necessary, our automation does that along with adding the names of those who approved the change. It doesn't hurt to add it manually, but you run into the risk of doing a typo. |
Landed in 3d0a317 |
PR-URL: nodejs#49284 Signed-off-by: Stewart X Addison <sxa@redhat.com> Fixes: nodejs#49278 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Simple doc change to add riscv64 to list of things that
os.arch()
andprocess.arch
can return.Fixes: #49278