Closed leonardocustodio closed 1 month ago
โฑ๏ธ Estimated effort to review: 1 ๐ตโชโชโชโช |
๐งช No relevant tests |
๐ No security concerns identified |
โก No key issues to review |
Category | Suggestion | Score |
Best practice |
Pin the decoder image version to include the patch number for better stability and predictability___ **To ensure the decoder service uses a specific version of the platform-decoder image,it's recommended to pin the version not only to the minor version but also to the patch version. This can help avoid unexpected behaviors due to automatic updates to newer patch versions which might not have been tested with your application.** [docker-compose.yml [31]](https://github.com/enjin/platform/pull/50/files#diff-e45e45baeda1c1e73482975a664062aa56f20c03dd9d64a827aba57775bed0d3R31-R31) ```diff -image: enjin/platform-decoder:v1.10.0 +image: enjin/platform-decoder:v1.10.0-patch ``` Suggestion importance[1-10]: 8Why: The suggestion to pin the image version to a specific patch version is a good practice for ensuring stability and predictability, as it prevents unexpected changes from automatic updates to newer patch versions. | 8 |
PR Type
enhancement, configuration changes
Description
docker-compose.yml
from versionv1.9.1
tov1.10.0
.v1.10.0
.Changes walkthrough ๐
docker-compose.yml
Upgrade decoder image version in Docker Compose
docker-compose.yml - Updated the decoder image version from `v1.9.1` to `v1.10.0`.