Skip to content

Conversation

iansu
Copy link
Contributor

@iansu iansu commented Feb 8, 2018

Update dotenv-expand to fix bug with environment variables that contain a $.

Closes #3961

@gaearon
Copy link
Contributor

gaearon commented Feb 8, 2018

It seems like we’ll want to do this against master too and cut another patch to 1.x.

@iansu
Copy link
Contributor Author

iansu commented Feb 8, 2018

Good idea. Should I submit another PR for that? Any idea why those tests are failing? Seems unrelated to this.

@gaearon
Copy link
Contributor

gaearon commented Feb 8, 2018

Yes, please send a PR.

@gaearon
Copy link
Contributor

gaearon commented Feb 8, 2018

Failing tests look worrying. cc @bradfordlemley

I had this symptom when Jest was "finding" a wrong version of babel-core (6 rather than 7).
This might mean something's wrong with how we test (or implement) monorepo support.

@bradfordlemley
Copy link
Contributor

Fix for the monorepo test failures: #3997. (Not related to the current PR.)

@gaearon gaearon added this to the 2.0.0 milestone Feb 9, 2018
@gaearon gaearon merged commit c263f17 into facebook:next Feb 9, 2018
1msoft added a commit to 1msoft/create-react-app that referenced this pull request Feb 10, 2018
akstuhl pushed a commit to akstuhl/create-react-app that referenced this pull request Mar 15, 2018
zmitry pushed a commit to zmitry/create-react-app that referenced this pull request Sep 30, 2018
@lock lock bot locked and limited conversation to collaborators Jan 20, 2019
@iansu iansu deleted the dotenv-expand-update branch October 18, 2019 05:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants