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

docs: remove reference to deprecated flag in npm-install #8026

Closed

Conversation

dellch
Copy link

@dellch dellch commented Jan 13, 2025

Replace --production with --omit=dev, and --production=false with --include=dev

Why

The --production flag is deprecated so we should avoid mentioning it when we can use --omit=dev instead:

production
Default: null
Type: null or Boolean
DEPRECATED: Use --omit=dev instead.

What

This PR helps discourage usage of deprecated commands by using:

  • --omit=dev instead of --production, and
  • --include=dev instead of --production=false.

References

Related to #6987
Fixes #8025

Replace `--production` with `--omit=dev`, and `--production=false` with `--include=dev`
@wraithgar
Copy link
Member

wraithgar commented Jan 13, 2025

This had been discussed in the past and it was reverted back to the current reference. What is more likely to happen is that --production is not going to be deprecated. There are some flags that are too old, and too many folks use them.

@wraithgar wraithgar closed this Jan 13, 2025
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 this pull request may close these issues.

[DOCS] npm install references deprecated flag --production
2 participants