Add "npm ci" support to npm module & state

[ralish]

Description of Issue/Question

For installing Node.js modules Salt provides the npm state which provides support for installing packages via a package.json file. Internally this results in Salt calling npm install. Recent versions of npm include the npm ci command which is far better suited to continuous integration and deployment scenarios as it ensures reproducible builds.

Introducing support should be relatively straightforward as the semantics are (near?) identical to npm install. For the npm state I propose adding a boolean option to the bootstrap function which will trigger usage of npm ci (and fail the state if it returns non-zero). In the npm module an equivalent boolean option should be implemented in the install function for the state to use.

I'm happy to take a stab at implementing support when I find some spare time if someone else doesn't beat me to it.

References:

• Introducing npm ci for faster, more reliable builds
• What is the difference between “npm install” and “npm ci”?

[dwoz]

@ralish Thanks, Looking forward to seeing your PR. :)

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

[ralish]

Please keep open. This is an important feature request and one I'd like to work on eventually.

[stale[bot]]

Thank you for updating this issue. It is no longer marked as stale.