Skip to content

Allow loading .env files + .env.deploy.<env> files - #342

Merged
ghedamat merged 1 commit into
ember-cli-deploy:masterfrom
blimmer:feature/use-default-dotenv-files
Jan 10, 2016
Merged

ghedamat merged 1 commit into
ember-cli-deploy:masterfrom
blimmer:feature/use-default-dotenv-files

Conversation

@blimmer

@blimmer blimmer commented Jan 9, 2016

Copy link
Copy Markdown
Contributor

Fixes #341.

Comment thread lib/tasks/read-config.js

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this comment here because the behavior in dotenv seem backwards to me. I would expect overrides to be loaded last!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

interesting, yeah I think dotenv discards already defined vars so you can do overrides using the process ENV

@ghedamat

ghedamat commented Jan 9, 2016

Copy link
Copy Markdown
Collaborator

@blimmer LGTM, unless @achambers or @lukemelia have anything to add I think we're good

could you squash the commits please?

Finally once we merge it would be ideal to add this to the guides, if you're up for it a PR against the temporary branch for 0.6.X docs would be amazing https://github.com/ember-cli/ember-cli-deploy/tree/gh-pages-0.6.x-first-pass

@blimmer

blimmer commented Jan 10, 2016

Copy link
Copy Markdown
Contributor Author

Squash is complete. I will start working on the PR against the docs. Thanks!

@blimmer
blimmer force-pushed the feature/use-default-dotenv-files branch from b78887c to c91b49a Compare January 10, 2016 17:13
@lukemelia

Copy link
Copy Markdown
Contributor

👍

ghedamat added a commit that referenced this pull request Jan 10, 2016
Allow loading .env files + .env.deploy.<env> files
@ghedamat
ghedamat merged commit b71e122 into ember-cli-deploy:master Jan 10, 2016
@ghedamat

Copy link
Copy Markdown
Collaborator

Thanks!

lukemelia pushed a commit that referenced this pull request Jan 22, 2016
To document the behavior introduced in #342.
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