Skip to content

Add support for dotenv#59

Open
LinusU wants to merge 3 commits into
hughsk:masterfrom
LinusU:dotenv
Open

Add support for dotenv#59
LinusU wants to merge 3 commits into
hughsk:masterfrom
LinusU:dotenv

Conversation

@LinusU
Copy link
Copy Markdown

@LinusU LinusU commented Aug 29, 2017

This PR adds support for loading a .env (dotenv) file containing additional environmental variables. It's super useful when you have multiple projects since it allows you to keep your environment contained to each project instead of having it global.

The file will only be loaded when the dotenv flag is passed, and variables from the file will only be used if they aren't already specified elsewhere which is how the dotenv package usually works when loading it straight into process.env.

I tried to follow the existing code style, please let me know if you need anything more. Thanks 👋

@LinusU
Copy link
Copy Markdown
Author

LinusU commented Sep 13, 2017

ping @hughsk, would love to see this in since I'm currently using this to build my production app 🙈 😄

@LinusU
Copy link
Copy Markdown
Author

LinusU commented Mar 15, 2018

ping @hughsk / @yoshuawuyts, happy to receive feedback ☺️

@alanaasmaa
Copy link
Copy Markdown

Started using @LinusU branch also. I will notify from my side when i get some problems.
Looking forward for a merge also :)

@LinusU
Copy link
Copy Markdown
Author

LinusU commented Aug 14, 2018

ping @hughsk / @yoshuawuyts, happy to receive feedback ☺️

@alanaasmaa
Copy link
Copy Markdown

Used it 4 months and no issues.

@LinusU
Copy link
Copy Markdown
Author

LinusU commented Nov 22, 2018

ping @hughsk / @yoshuawuyts, happy to receive feedback ☺️

@yoshuawuyts
Copy link
Copy Markdown
Collaborator

@LinusU This module is considered feature-complete, which means adding more features isn't within the scope of the project. We should probably write a note about that somewhere though.

Pretty much the only patches that would be accepted at this point are to fix bugs with versions of browserify, or other incompatibility issues. It would probably be best to release a separate module for dotenv support. Thanks!

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.

3 participants