Merge pull request #222 from Automattic/remove-vars

fix(mariadb): remove unused variables
Automattic · Jun 1, 2024 · e624d88 · e624d88
2 parents 272a889 + 1002f27
commit e624d88
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/features/src/mariadb/devcontainer-feature.json b/features/src/mariadb/devcontainer-feature.json
@@ -1,7 +1,7 @@
 {
     "name": "MariaDB",
     "id": "mariadb",
-    "version": "1.1.2",
+    "version": "1.1.3",
     "description": "Sets up MariaDB into the Dev Environment",
     "options": {
         "installDatabaseToWorkspaces": {

diff --git a/features/src/mariadb/service-run.tpl b/features/src/mariadb/service-run.tpl
@@ -8,9 +8,7 @@ if [ -f /etc/conf.d/mariadb ]; then
     . /etc/conf.d/mariadb
 fi
 
-: "${MARIADB_USER:=mysql}"
-: "${MARIADB_DATADIR:=/var/lib/mysql}"
-
+# shellcheck disable=SC2154
 install -d -D -m 02755 -o "${MARIADB_USER}" -g "${MARIADB_USER}" "${MARIADB_DATADIR}"
 chown -R "${MARIADB_USER}:${MARIADB_USER}" "${MARIADB_DATADIR}"