You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Title. Can't find another issue about it, so maybe it wasn't asked before? (seems unlikely but idk)
Especially when dealing with docker images it's significantly easier to work with environment variables than command arguments, due to how ENTRYPOINT + CMD + arguments work together being somewhat inconsistent from runtime to runtime.
Seems like argparse (which is the method used here afaik) doesn't natively handle environment variables for some reason; I do quite like this approach as a way to do it, although there are a few other methods.
I'd happily make a PR, but I figured I'd just suggest it first in case it's been asked and rejected before (but not on Github so I couldn't find it with a search).
Worst case scenario if it's unwanted, the next person to think about this sees this issue and can directly know so.
The text was updated successfully, but these errors were encountered:
Title. Can't find another issue about it, so maybe it wasn't asked before? (seems unlikely but idk)
Especially when dealing with docker images it's significantly easier to work with environment variables than command arguments, due to how ENTRYPOINT + CMD + arguments work together being somewhat inconsistent from runtime to runtime.
Seems like argparse (which is the method used here afaik) doesn't natively handle environment variables for some reason; I do quite like this approach as a way to do it, although there are a few other methods.
I'd happily make a PR, but I figured I'd just suggest it first in case it's been asked and rejected before (but not on Github so I couldn't find it with a search).
Worst case scenario if it's unwanted, the next person to think about this sees this issue and can directly know so.
The text was updated successfully, but these errors were encountered: