-
Notifications
You must be signed in to change notification settings - Fork 16
extracted timezone env from composers into .env file #4
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
Conversation
Thank you for contribution. Add default value of TZ for back compatibility please.
540a8c8 to
d4f9013
Compare
Hi, what do You mean by default value of TZ. I thought that TZ=Europe/Moscow in .env_example file is enough. I must point out that I'm new into open-source contributing so I'm still learning 😄
I tell about this case:
- I already have composition
- I already have copy of .env_example
- I pull new changes
- I have empty TZ instead of old value 'Europe/Moscow'
I think we need this
environment:
 TZ: "${TZ:-Europe/Moscow}"
Can you check how to work it please?
Oh okay i didn't know about default variables in docker envs. I will edit the pr tommorow
Thank you for contribution. Add default value of TZ for back compatibility please.
Done.
Thank you!
That changes allows user to change timezone quickly in .env file and also it standardize same timezone across all containers