You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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.
Description of Issue/Question
For installing Node.js modules Salt provides the
npm
state which provides support for installing packages via apackage.json
file. Internally this results in Salt callingnpm install
. Recent versions ofnpm
include thenpm 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 thenpm
state I propose adding a boolean option to thebootstrap
function which will trigger usage ofnpm ci
(and fail the state if it returns non-zero). In thenpm module
an equivalent boolean option should be implemented in theinstall
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:
npm ci
for faster, more reliable buildsThe text was updated successfully, but these errors were encountered: