feat(release)!: Use the new nodesource repository instead of the old …

…installation script 🧓
gremo · Sep 14, 2023 · 0f7e842 · 0f7e842
1 parent e1667f6
commit 0f7e842
Show file tree

Hide file tree

Showing 6 changed files with 22 additions and 9 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -9,7 +9,7 @@ ARG COMPOSER_VERSION=lts
 #ejs <% if (production) { _%>
 ARG KNOWN_HOSTS="bitbucket.org github.com gitlab.com"
 #ejs <% } _%>
-ARG NODE_VERSION=lts
+ARG NODE_VERSION=18
 #ejs <% if (mariadb) { _%>
 ARG MARIADB_VERSION=11
 #ejs <% } _%>
@@ -154,7 +154,10 @@ RUN \
         install-php-extensions $EXTENSIONS; \
     fi; \
     # Install Node.js, update npm and install Yarm
-    curl -fsSL https://deb.nodesource.com/setup_"$NODE_VERSION".x | bash -; \
+    mkdir -p /etc/apt/keyrings; \
+    curl -fsSL https://deb.nodesource.com/gpgkey/nodesource-repo.gpg.key | gpg --dearmor -o /etc/apt/keyrings/nodesource.gpg; \
+    echo "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] https://deb.nodesource.com/node_$NODE_VERSION.x nodistro main" | tee /etc/apt/sources.list.d/nodesource.list; \
+    apt-get update; \
     apt-get install -y --no-install-recommends nodejs; \
     npm update -g npm; \
     npm install -g yarn; \

diff --git a/UPGRADE.md b/UPGRADE.md
@@ -1,5 +1,12 @@
 # Upgrade
 
+## `0.2.x` to `0.3.0`
+
+The PHP development image now uses the [new installation method for node](https://github.com/nodesource/distributions). Long story short, `lts` and `current` keywords can't be used anymore in `NODE_VERSION` variable.
+
+- Pull the updated PHP development image
+- Carefully merge changes in `Dockerfile`, `docker-compose.yml`, `docker-compose.override.yml`
+
 ## `0.2.x` to `0.2.4`
 
 - In your `docker-compose.dev.yml` ensure `caddy.volumes` contains `public_bundles:/srv/public/bundles`

diff --git a/docker-compose.override.yml b/docker-compose.override.yml
@@ -7,7 +7,7 @@ services:
                 PHP_VERSION: ${PHP_VERSION:-8}
                 PHP_EXTENSIONS: ${PHP_EXTENSIONS:-}
                 COMPOSER_VERSION: ${COMPOSER_VERSION:-lts}
-                NODE_VERSION: ${NODE_VERSION:-lts}
+                NODE_VERSION: ${NODE_VERSION:-18}
         environment:
             APP_ENV: prod
         ports:
@@ -27,7 +27,7 @@ services:
         build:
             target: php-prod
             args:
-                NODE_VERSION: ${NODE_VERSION:-lts}
+                NODE_VERSION: ${NODE_VERSION:-18}
         environment:
             APP_ENV: prod
         volumes:

diff --git a/docker-compose.yml b/docker-compose.yml
@@ -25,7 +25,7 @@ services:
                 PHP_VERSION: ${PHP_VERSION:-8}
                 PHP_EXTENSIONS: ${PHP_EXTENSIONS:-}
                 COMPOSER_VERSION: ${COMPOSER_VERSION:-lts}
-                NODE_VERSION: ${NODE_VERSION:-lts}
+                NODE_VERSION: ${NODE_VERSION:-18}
         #ejs <%_ if (!mariadb) { _%>
         environment:
             DATABASE_ENABLE_MIGRATIONS: false

diff --git a/docker/php/Dockerfile b/docker/php/Dockerfile
@@ -83,6 +83,9 @@ ARG EXTENSIONS_DEV
 COPY --link php.dev.ini /usr/local/etc/php/conf.d/symfony-sail.dev.ini
 
 RUN \
+    # Install OS packages
+    apt-get update; \
+    apt-get install -y --no-install-recommends gnupg; \
     # Copy base configuration file
     cp "$PHP_INI_DIR"/php.ini-development "$PHP_INI_DIR"/php.ini; \
     # Install PHP extensions

diff --git a/docs/configuration.md b/docs/configuration.md
@@ -120,7 +120,7 @@ Summary of variables, where they should be defined, and their default values:
       <td><code>NODE_VERSION</code></td>
       <td align="center">✅</td>
       <td align="center">❌</td>
-      <td>lts</td>
+      <td>18</td>
     </tr>
     <tr>
       <td colspan="4" align="center">
@@ -231,9 +231,9 @@ Further considerations:
 
 ### Node-related variables
 
-| Variable       | Allowed values        | Notes |
-| :------------- | :-------------------- | :---- |
-| `NODE_VERSION` | "current", "lts", `x` |       |
+| Variable       | Allowed values | Notes |
+| :------------- | :------------- | :---- |
+| `NODE_VERSION` | `x`            |       |
 
 ### Database-related variables