Skip to content

Commit 3fe77d5

Browse files
authored
doc: fix the fnm docs for a functional instruction guide (#6919)
Before the change: when the commands were copy&paste (followed to the letter) they wouldn't work and users wouldn't have a functional node or npm binary After the change: correct the instructions so that the fnm library is first sourced from the bash shell to activate it into the path which allows to use fnm as an executable and run node and npm binaries too. Signed-off-by: Liran Tal <[email protected]>
1 parent 4a53573 commit 3fe77d5

2 files changed

Lines changed: 4 additions & 0 deletions

File tree

apps/site/i18n/locales/en.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -300,6 +300,7 @@
300300
"shouldPrint": "should print `{version}`",
301301
"installsFnm": "installs fnm (Fast Node Manager)",
302302
"downloadAndInstallNodejs": "download and install Node.js",
303+
"activateFNM": "activate fnm",
303304
"noteWithColon": "NOTE:",
304305
"dockerIsNotNodejsPackageManager": "Docker is not a Node.js package manager.",
305306
"PleaseEndureAlreadyInstallOnSystem": "Please ensure it is already installed on your system.",

apps/site/util/getNodeDownloadSnippet.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,9 @@ export const getNodeDownloadSnippet = (
5454
# ${t('layouts.download.codeBox.installsFnm')}
5555
curl -fsSL https://fnm.vercel.app/install | bash
5656
57+
# ${t('layouts.download.codeBox.activateFNM')}
58+
source ~/.bashrc
59+
5760
# ${t('layouts.download.codeBox.downloadAndInstallNodejs')}
5861
fnm use --install-if-missing ${release.major}
5962

0 commit comments

Comments
 (0)