Upgrade dotenv and dotenv-expand to latest version#7119
Conversation
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need the corporate CLA signed. If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks! |
1e95a1e to
a27b63b
Compare
|
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks! |
|
It's not really clear what the breaking changes are in |
|
@iansu I believe the The |
|
Looks like this was already done in another PR. Thanks :) |
The
5.*.*release ofdotenv-expandfixes a bug where ENV variables set in the CLI as empty were getting overwritten by values in.envfiles, this bug was previously fixed indotenv, but the fix was not ported over todotenv-expanduntil recently.It also adds better support for ENV variable values with
$s in them and better handling of escaped$s in values.I went ahead and updated
dotenvto the latest version, as well. I believe the major version bump was just to drop support for older versions of Node.