Closed ekzyis closed 2 months ago
Arrays for env_file are only supported in Docker Compose >=v2.24 which is too new (from January 2024). Most distros distribute older packages.
env_file
Since --env-file as defined in the sndev script acts as an override for env_file anyway, we can safely remove it here.
--env-file
See https://github.com/stackernews/stacker.news/pull/1082#issuecomment-2081230695 and https://github.com/stackernews/stacker.news/commit/dd6e921e2e00950f19da5bd6d76769c89268d1bb#r141433414
[x] Did you QA this? Could we deploy this straight to production?
[ ] For frontend changes: Tested on mobile?
[ ] Did you introduce any new environment variables? If so, call them out explicitly in the PR description.
Description
Arrays for
env_fileare only supported in Docker Compose >=v2.24 which is too new (from January 2024). Most distros distribute older packages.Since
--env-fileas defined in the sndev script acts as an override forenv_fileanyway, we can safely remove it here.Additional Context
See https://github.com/stackernews/stacker.news/pull/1082#issuecomment-2081230695 and https://github.com/stackernews/stacker.news/commit/dd6e921e2e00950f19da5bd6d76769c89268d1bb#r141433414
Checklist
[x] Did you QA this? Could we deploy this straight to production?
[ ] For frontend changes: Tested on mobile?
[ ] Did you introduce any new environment variables? If so, call them out explicitly in the PR description.