From 354d20147688c2670a4d5c75ff94d1c79b6d79e9 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sun, 7 Dec 2025 13:25:31 +0000 Subject: [PATCH] Add shellcheck disable comment for intentional word splitting Co-authored-by: jbtronics <5410681+jbtronics@users.noreply.github.com> --- .docker/frankenphp/docker-entrypoint.sh | 3 +++ .docker/partdb-entrypoint.sh | 3 +++ 2 files changed, 6 insertions(+) diff --git a/.docker/frankenphp/docker-entrypoint.sh b/.docker/frankenphp/docker-entrypoint.sh index ff1fbb69..0aaf6674 100644 --- a/.docker/frankenphp/docker-entrypoint.sh +++ b/.docker/frankenphp/docker-entrypoint.sh @@ -29,6 +29,9 @@ if [ "$1" = 'frankenphp' ] || [ "$1" = 'php' ] || [ "$1" = 'bin/console' ]; then # Install additional composer packages if COMPOSER_EXTRA_PACKAGES is set if [ -n "$COMPOSER_EXTRA_PACKAGES" ]; then echo "Installing additional composer packages: $COMPOSER_EXTRA_PACKAGES" + # Note: COMPOSER_EXTRA_PACKAGES is intentionally not quoted to allow word splitting + # This enables passing multiple package names separated by spaces + # shellcheck disable=SC2086 composer require $COMPOSER_EXTRA_PACKAGES --no-interaction --no-progress --optimize-autoloader if [ $? -eq 0 ]; then echo "Successfully installed additional composer packages" diff --git a/.docker/partdb-entrypoint.sh b/.docker/partdb-entrypoint.sh index 94d7f0f6..745fccd4 100644 --- a/.docker/partdb-entrypoint.sh +++ b/.docker/partdb-entrypoint.sh @@ -42,6 +42,9 @@ fi # Install additional composer packages if COMPOSER_EXTRA_PACKAGES is set if [ -n "$COMPOSER_EXTRA_PACKAGES" ]; then echo "Installing additional composer packages: $COMPOSER_EXTRA_PACKAGES" + # Note: COMPOSER_EXTRA_PACKAGES is intentionally not quoted to allow word splitting + # This enables passing multiple package names separated by spaces + # shellcheck disable=SC2086 sudo -E -u www-data composer require $COMPOSER_EXTRA_PACKAGES --no-interaction --no-progress --optimize-autoloader if [ $? -eq 0 ]; then echo "Successfully installed additional composer packages"