Fix Docker warnings#14440
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the env_file path interpolation in docker-compose.yml to dynamically resolve environment files using the ENVIRONMENT variable. However, Docker Compose does not support the :+ (alternate value) operator, which will trigger warnings and lead to incorrect file resolution (e.g., .envdev instead of .env_dev). It is recommended to use a dedicated ENV_FILE variable with a fallback default, such as ${ENV_FILE:-.env}, to ensure compatibility and cleaner configuration.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| container_name: nginx4${COMPOSE_PROJECT_NAME} | ||
| env_file: | ||
| - .env${ENVIRONMENT:+_${ENVIRONMENT}} | ||
| - .env${ENVIRONMENT:+_}${ENVIRONMENT:-} |
There was a problem hiding this comment.
As noted in the comment on line 9, the :+ operator is not supported by Docker Compose interpolation. Please consider using the ENV_FILE pattern here as well to avoid warnings and ensure correct file resolution.
- ${ENV_FILE:-.env}| container_name: letsencrypt4${COMPOSE_PROJECT_NAME} | ||
| env_file: | ||
| - .env${ENVIRONMENT:+_${ENVIRONMENT}} | ||
| - .env${ENVIRONMENT:+_}${ENVIRONMENT:-} |
There was a problem hiding this comment.
As noted in the comment on line 9, the :+ operator is not supported by Docker Compose interpolation. Please consider using the ENV_FILE pattern here as well to avoid warnings and ensure correct file resolution.
- ${ENV_FILE:-.env}
2 participants
No description provided.