-
-
Notifications
You must be signed in to change notification settings - Fork 68
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
feat: Add --recursive flag to enable env-var nesting #253
base: master
Are you sure you want to change the base?
Conversation
Look like this matches an existing feature request: #97 |
Also possibly #122 |
awesome to see this supported out of the box 🎉 |
@toddbluhm Hello! It would be great if you could merge it. Variable expansion is essential for most complex projects |
this would be really useful.... |
@toddbluhm - back to a project where this would be really useful. |
@nkrul you still interested in this? If your willing to refactor this a tad bit into the current |
Added a command line
--recursive
option.This will reprocess the env file to fill in any ${variables} there may be.
eg:
will resolve VAL2 to
Hello There
It's not order dependent, as long as it's in the file
It's all done in a single pass.
Also made
$var
and${var}
both workAlso fixed a return type bug (process.kill can return a boolean in my version of node)
This has been tested on a real project with a file:/ dependency include;