-
Notifications
You must be signed in to change notification settings - Fork 74
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
Parameters from ENV #63
Comments
I'm not the biggest fan of parameterization using env vars. That said, doing it is very simple; you just pass an implementation of |
Does the Also, what about the Also, why aren't you a fan? I'm interested to know. |
Yeah, which is odd. Probably just blindly replicating ConfigurationManager
behaviour.
…On Wed, 7 Dec 2016 at 09:15, Henrik Feldt ***@***.***> wrote:
Does the GetValue : string -> string return null if it doesn't have it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#63 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ACrts2O1xk0dIW6eLjtlCHeoL2ZiXClCks5rFnlCgaJpZM4LEi21>
.
|
You got a PR from me! |
@haf It's just that I think using env vars for parameterization is ugly, violates referential transparency considerations and can be potentially dangerous. If completely unavoidable, I would rather just dereference the vars at the script level and pass through the CLI:
If I am going depend on ambient state for configuration I would much rather have the ability to explicitly specify which aspects of that ambient state I want to use. Suppose you are consuming an application that I wrote which uses Argu for environment variable parsing. Unbeknownst to you, my application is also looking for a The above example by the way is inspired by a true story. |
Description
I'd like to use Argu to optionally take configuration from the environment. Do you think you could add that feature?
Currently writing a networked-linux tutorial and figured I wanted to showcase how to build micro-services with F#.
The text was updated successfully, but these errors were encountered: