Skip to content
This repository has been archived by the owner on Aug 19, 2022. It is now read-only.

Clarify in README that dotenv gem is still required #7

Closed
pelargir opened this issue May 8, 2014 · 4 comments
Closed

Clarify in README that dotenv gem is still required #7

pelargir opened this issue May 8, 2014 · 4 comments

Comments

@pelargir
Copy link

pelargir commented May 8, 2014

I was confused for a bit because my .env file wasn't being loaded. I assumed the dotenv gem was a dependency of the dotenv-deployment gem but I guess it wasn't. You should clarify in the README that both dotenv-deployment AND dotenv have to be added to the Gemfile before things work as expected.

@markottaviani
Copy link

+1 had the exact same problem.

@kntsoriano
Copy link

Had the same problem also. This has to be clarified in the readme.

markottaviani added a commit to markottaviani/dotenv-deployment that referenced this issue May 20, 2014
Clarified in README that dotenv gem is still required for issue bkeepers#7
@djmaze
Copy link

djmaze commented May 30, 2014

Shouldn't there be a runtime dependency in the gemspec, so this will work automatically?

@bkeepers
Copy link
Owner

bkeepers commented Jun 2, 2014

The next release will have the dependencies declared properly. I had to do it this way so it didn't break for people upgrading from previous version.

@bkeepers bkeepers closed this as completed Jun 2, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants