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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to raise error, if .env file does not exist #1 #2

Merged
merged 6 commits into from Feb 11, 2018

Conversation

thegreyfellow
Copy link
Contributor

This is in response to the issue I opened #1 , please let me know your thoughts about it 馃檹

src/envy.cr Outdated
def {{name.id}}(hash : Hash(String, String))
hash.each do |key, value|
ENV[key] {{comp.id}} value
end
end
{% end %}

def parse(filename : String) : Hash(String, String)
def parse(filename : String, hash = {} of Symbol => Bool) : Hash(String, String)
Copy link
Owner

Choose a reason for hiding this comment

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

Calling it option would be more descriptive than hash.

@petoem
Copy link
Owner

petoem commented Feb 11, 2018

I love it.
Using a Hash makes it even possible to pass additional options to parse.

@thegreyfellow
Copy link
Contributor Author

Awesome!! I'll be pushing that update right now.

@petoem
Copy link
Owner

petoem commented Feb 11, 2018

Great!
Closes #1

@petoem petoem merged commit 1e41632 into petoem:master Feb 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants