Page MenuHomeSoftware Heritage

Update getting-started to use nodesource's package repository to install npm. Fixes T1225.
ClosedPublic

Authored by vlorentz on Oct 2 2018, 2:39 PM.

Diff Detail

Repository
rDDOC Development documentation
Branch
install-npm
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 1477
Build 1821: arc lint + arc unit

Event Timeline

Apart the removal of npm from the apt install command line, this can be landed.

docs/getting-started.rst
52

the npm package is not needed here as it comes with the nodejs package from nodesource

This revision was not accepted when it landed; it landed in state Needs Review.Oct 2 2018, 3:53 PM
This revision was automatically updated to reflect the committed changes.
vlorentz added inline comments.
docs/getting-started.rst
52

The curl|bash only adds repositories, it does not install npm itself.

docs/getting-started.rst
52

You did not read correctly. npm is bundled in the nodejs package so there is no need to add npm to the apt install command line (because nodesource only provide a nodejs package).

vlorentz added inline comments.
docs/getting-started.rst
52

Oops, I missed that last comment. Fixed in 3d29d4a45f449899da5482608e756b866b43db07.