-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
Env var settings #197
base: master
Are you sure you want to change the base?
Env var settings #197
Conversation
02e32fc
to
0d3914b
Compare
0d3914b
to
ded6a5c
Compare
@zorchenhimer : For that PR, I have a question. What is the overloading order you want for the parameter? Right now, thanks to the Would you prefer to use the |
It should check in this order. If one isn't assigned, move to the next.
|
Do you mean that : |
Yes. |
Okay, for that beheave I'll have to refactor a lot of the params loading. I'm on it. |
@zorchenhimer : I guess you don't have anything against Object Oriented Style of programming ? I'll need/use that for the parameter loading. |
@zorchenhimer : I'm not resigning on this, I'm just missing time nowadays. |
They @zorchenhimer ! I's like to suggest, for the parameters of MovieNight, With a loading and priority order like :
|
No description provided.