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

Enforce NODE_ENV=test #5020

Merged
merged 1 commit into from
Oct 1, 2015
Merged

Conversation

sebmarkbage
Copy link
Collaborator

You could make the argument that this should be optional, but it doesn't
work without it so we might as well just enforce it.

Makes jest work by default.

You could make the argument that this should be optional, but it doesn't
work without it so we might as well just enforce it.

Makes `jest` work by default.
@sophiebits
Copy link
Collaborator

Sure.

@zpao
Copy link
Member

zpao commented Oct 1, 2015

for later: it's definitely possible to change the transforms so they look at an argument instead of process.env

sebmarkbage added a commit that referenced this pull request Oct 1, 2015
@sebmarkbage sebmarkbage merged commit 401e6f1 into facebook:master Oct 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants