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
Remove redundant NODE_ENV=test in Makefile #5350
Conversation
`NODE_ENV=test` is exported by default. Therefore, targets that set `NODE_ENV` to `test` are cleaned up.
@aaronang, thanks for your PR! By analyzing the history of the files in this pull request, we identified @hzoo, @kaicataldo and @danez to be potential reviewers. |
Codecov Report
@@ Coverage Diff @@
## master #5350 +/- ##
=========================================
Coverage ? 89.44%
=========================================
Files ? 204
Lines ? 9949
Branches ? 2689
=========================================
Hits ? 8899
Misses ? 1050
Partials ? 0 Continue to review full report at Codecov.
|
make test-only | ||
|
||
test-ci-coverage: | ||
NODE_ENV=test BABEL_ENV=cov make bootstrap |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that is odd, since BABEL_ENV is set here.
const envKey = process.env.BABEL_ENV || process.env.NODE_ENV || "development";
in our code currently
Good catch! |
`NODE_ENV=test` is exported by default. Therefore, targets that set `NODE_ENV` to `test` are cleaned up.
NODE_ENV=test
is exported by default inMakefile
. Therefore, targets that setNODE_ENV
totest
are cleaned up.