babel / babel/minify

babel-plugin-transform-remove-console is not run for production where it matters

Open
#1,055 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
4.4k
Forks
217
PR merge metrics
No merged PRs in 30d

Description

looking at the example it says:

`npm install babel-plugin-transform-remove-console --save-dev`

Which means it will be saved to devdependencies. It will not be run in production is NODE_ENV is set to production. I think the installation example is little bit bad because of this. If you set `NODE_ENV=production` it will not install and run devDependencies at all.

I think the installation instructions should be changed to:

`npm install babel-plugin-transform-remove-console`

Please do correct me if I'm wrong.
Or is this not meant for production use? Maybe state that then.

Contributor guide

Open the contributing guide

Research direction

Inspect the example installation instructions referenced in the issue and verify how the plugin is intended to be used when NODE_ENV is production. Update the documented install command or clarify that production use is not intended, with the example accurately describing the supported setup.

Written by the indexing model from the issue text.

Assessment

Tech stack
babel, javascript
Domain
documentation
Issue type
Documentation
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.