-
Notifications
You must be signed in to change notification settings - Fork 34
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
No default and nil as default aren't the same #49
Comments
@janpieper good catch, feel free to send a PR to fix the documentation. It should raise indeed because often if you don't have defaults you won't want to end up loading invalid configuration. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Maybe I am wrong, but this doesn't feel to be the truth:
confex/lib/confex.ex
Line 52 in fa436d1
For me, I get an
ArgumentError
when using{:system, :integer, "ENV_NAME"}
:But it works fine when using
{:system, :integer, "ENV_NAME", nil}
. So, it is not the same.The text was updated successfully, but these errors were encountered: