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.

Sign up for GitHub

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

Jump to bottom

Remove envy dependency #20

Open

benbro opened this issue Dec 6, 2015 · 0 comments

benbro commented Dec 6, 2015

The envy app doesn't add real value.
It is only used in a single place and can easily replaced.
I suggest to remove it to simplify the dependencies.

Defaults = envy:get(mini_s3, s3_defaults, list),

Defaults = case application:get_env(mini_s3, s3_defaults) of
    Def when is_list(Def) -> Def;
    _ -> []
end,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment