-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Support dotenv - New feature request #7056
Comments
@crazyoptimist Thanks for posting! We'll take a look as soon as possible. In the mean time, there are a few ways you can help speed things along:
Please remember: never post in a public forum if you believe you've found a genuine security vulnerability. Instead, disclose it responsibly. For help with questions about Sails, click here. |
Thanks for the feedback @crazyoptimist. I think you're right about implementing it as a hook, If you're feeling up to it you could make it an installable hook to make this available to other Sails users. https://sailsjs.com/documentation/concepts/extending-sails/hooks/installable-hooks#creating-an-installable-hook |
Okay, |
Bang! |
Let me close this issue for now assuming you have |
Thanks for your efforts, maintainers and contributors.
Current implementation can not leverage dotenv@npm in case of using sails cli.
I guess it can be solved by a hook using dotenv npm package as a dependency.
Let me know your opinion guys.
The text was updated successfully, but these errors were encountered: