[BUGFIX] Remove notes from shellcheck, too

Fixes #1398
TYPO3BestPractices · Jul 30, 2024 · 53b67d4 · 53b67d4
1 parent 8d80ad0
commit 53b67d4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -313,6 +313,9 @@ jobs:
         uses: actions/checkout@v4
       - name: Check scripts
         uses: ludeeus/action-shellcheck@master
+        env:
+          # ignore "double-quoted" warnings to stay close to the TYPO3 core
+          SHELLCHECK_OPTS: -e SC2086
   documentation:
     name: Documentation
     runs-on: ubuntu-24.04

diff --git a/Build/Scripts/runTests.sh b/Build/Scripts/runTests.sh
@@ -317,7 +317,7 @@ CONTAINER_INTERACTIVE="-it --init"
 HOST_UID=$(id -u)
 HOST_PID=$(id -g)
 USERSET=""
-SUFFIX=$(echo $RANDOM)
+SUFFIX="$RANDOM"
 NETWORK="friendsoftypo3-tea-${SUFFIX}"
 CI_PARAMS="${CI_PARAMS:-}"
 CONTAINER_HOST="host.docker.internal"