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

chore: upgrade dotenv and npm audit fix #827

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

naseemkullah
Copy link

The main reason for this PR is actually to support new features in dotenv related to config via env vars.

see https://github.com/motdotla/dotenv#preload for examples of DOTENV_CONFIG_* env vars.

Note, this makes the use of the require('rc')('db-migrate') added via aef82c3 somewhat redundant, since DOTENV_CONFIG_PATH is one of the env vars supported by dotenv.

The main reason for this PR is actually to support new features in dotenv
related to config via env vars.

see https://github.com/motdotla/dotenv#preload for examples of DOTENV_CONFIG_* env vars.

Note, this makes the use of the require('rc')('db-migrate') added via aef82c3 somewhat redundant,
since DOTENV_CONFIG_PATH is one of the env vars supported by dotenv.

Signed-off-by: Naseem Ullah <24660299+naseemkullah@users.noreply.github.com>
@naseemkullah
Copy link
Author

@wzrdtales friendly ping

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant