search

man-chi / example-antora-basic

MIT License
0 stars 2 forks source link

Update the npm install command #1

Closed mojavelinux closed 5 months ago

mojavelinux commented 5 months ago

Instead of this:

npm install -g asciidoctor asciidoctor-kroki @antora/cli @antora/site-generator @antora/lunr-extension

The install command should be:

npm install antora asciidoctor-kroki @antora/lunr-extension

There's no reason to install the modules globally. The Antora package should be consolidated to antora. There's also no reason to install the asciidoctor package as that is already included with Antora and the version installed independently will be incorrect.

man-chi commented 5 months ago

thanks for the comment, let me fix this today

man-chi commented 5 months ago

Instead of this:

npm install -g asciidoctor asciidoctor-kroki @antora/cli @antora/site-generator @antora/lunr-extension

The install command should be:

npm install antora asciidoctor-kroki @antora/lunr-extension

There's no reason to install the modules globally. The Antora package should be consolidated to antora. There's also no reason to install the asciidoctor package as that is already included with Antora and the version installed independently will be incorrect.

fixed in the new push.

mojavelinux commented 5 months ago

Excellent news!