Use minor version as Drupal constraint #18
Conversation
| # https://www.drupal.org/project/drupal/releases/10.1.2 | ||
| ENV DRUPAL_VERSION 10.1.2 | ||
| # https://www.drupal.org/project/drupal/releases | ||
| ENV DRUPAL_VERSION 10.1 | ||
|
|
||
| WORKDIR /opt/drupal | ||
| # Copy from the official image when Drupal 10.0 will be released |
There was a problem hiding this comment.
I really don't remember what this refers to 😂
Not sure if Drupal releases any official Docker images.
There was a problem hiding this comment.
The official images are here: https://hub.docker.com/_/drupal
Something like COPY --from=drupal ---link /opt/drupal /opt/drupal should do the trick and be always up to date.
(https://github.com/docker-library/drupal/blob/master/Dockerfile.template)
| @@ -9,8 +9,6 @@ | |||
| # Database password: example | |||
| # ADVANCED OPTIONS; Database host: postgres | |||
|
|
|||
| version: '3.1' | |||
|
|
|||
| services: | |||
There was a problem hiding this comment.
We should also rename the file compose.yaml to be in sync with be Docker best practices.
There was a problem hiding this comment.
Can do, is there some documentation for this? This is a new thing to me.
There was a problem hiding this comment.
Yes: https://docs.docker.com/compose/compose-file/03-compose-file/
Just renaming the file should be enough.
There was a problem hiding this comment.
Hmm, for me any docker compose command will result with no configuration file provided: not found after renaming the file. Docker Compose version v2.22.0
|
Thank you for your contribution @back-2-95. |
|
@dunglas hi, I can become co-maintainer. This is quite simple repo. That said, I'd like to set some goals: Crystallising the purpose:
Providing some quickstart for Drupal community:
FYI: I don't actively code Drupal sites myself, I mostly provide tooling and devops for Drupalists for their projects. |
Instead of installing a specific patch version of Drupal, install latest minor version.
Will resolve #19