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
Is there an existing issue that is already proposing this?
I have searched the existing issues
Is your feature request related to a problem? Please describe it
When installing, a regular npm install is made which could bump some versions during install. This makes for harder reproductions since installs aren't reproduceable.
Describe the solution you'd like
Install with the versions specified in the lock file. This would then translate to the following install commands.
We use nypm for installing dependencies since it has automatic package manager resolution. It does not currently have an option for using the lock file. I opened one here unjs/nypm#115
In the meantime, we can use execa to execute the command we want.
- install with "npm ci" instead of "npm install" so package-lock.json is respected
- implement a detected package manager to install command map to preserve behavior of nypm's installDependencies command and prepare for when we want to choose between package managers
- leave a link to the open issue for adding this feature into nypm
Closes#24
Is there an existing issue that is already proposing this?
Is your feature request related to a problem? Please describe it
When installing, a regular
npm install
is made which could bump some versions during install. This makes for harder reproductions since installs aren't reproduceable.Describe the solution you'd like
Install with the versions specified in the lock file. This would then translate to the following install commands.
pnpm install --frozen-lockfile
https://pnpm.io/cli/install#--frozen-lockfileyarn install --forzen-lockfile
https://classic.yarnpkg.com/lang/en/docs/cli/install/#toc-yarn-install-frozen-lockfilenpm ci
https://docs.npmjs.com/cli/v10/commands/npm-ciWe use
nypm
for installing dependencies since it has automatic package manager resolution. It does not currently have an option for using the lock file. I opened one here unjs/nypm#115In the meantime, we can use
execa
to execute the command we want.Teachability, documentation, adoption, migration strategy
No response
What is the motivation / use case for changing the behavior?
This makes for easier reproductions and deterministic output from the cli.
The text was updated successfully, but these errors were encountered: