Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI workflows using removed npm install options #24

Open
MikeMcC399 opened this issue Dec 11, 2024 · 0 comments · May be fixed by #25
Open

CI workflows using removed npm install options #24

MikeMcC399 opened this issue Dec 11, 2024 · 0 comments · May be fixed by #25

Comments

@MikeMcC399
Copy link

Issue

The following CI workflows are using npm install options which do not work on Node.js >= 16 (npm >= 8):

The lowest supported Node.js version is v18.0.0, which bundles npm v8.6.0.

only=prod

Run npm install --only=prod
npm warn config only Use `--omit=dev` to omit dev dependencies from the install.

only=dev

Run npm install --only=dev
npm WARN invalid config only="dev" set in command line options
npm WARN invalid config Must be one of: null, prod, production

Since the repo contains no production dependencies to install, only devDependencies, logging the installation of the two types of dependencies separately seems to be overkill. (See package.json.)

Reference

https://docs.npmjs.com/cli/v6/commands/npm-install

The --only={prod[uction]|dev[elopment]} argument will cause either only devDependencies or only non-devDependencies to be installed regardless of the NODE_ENV.

https://docs.npmjs.com/cli/v10/using-npm/config#include

Default:
Type: "prod", "dev", "optional", or "peer" (can be set multiple times)

https://docs.npmjs.com/cli/v10/using-npm/config#omit

Default: 'dev' if the NODE_ENV environment variable is set to 'production', otherwise empty.
Type: "dev", "optional", or "peer" (can be set multiple times)

@MikeMcC399 MikeMcC399 linked a pull request Dec 12, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant