Add shellcheck disable comment for intentional word splitting

Co-authored-by: jbtronics <5410681+jbtronics@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot] 2025-12-07 13:25:31 +00:00
parent 43996160c0
commit 354d201476
2 changed files with 6 additions and 0 deletions

View file

@ -29,6 +29,9 @@ if [ "$1" = 'frankenphp' ] || [ "$1" = 'php' ] || [ "$1" = 'bin/console' ]; then
# Install additional composer packages if COMPOSER_EXTRA_PACKAGES is set # Install additional composer packages if COMPOSER_EXTRA_PACKAGES is set
if [ -n "$COMPOSER_EXTRA_PACKAGES" ]; then if [ -n "$COMPOSER_EXTRA_PACKAGES" ]; then
echo "Installing additional composer packages: $COMPOSER_EXTRA_PACKAGES" 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 composer require $COMPOSER_EXTRA_PACKAGES --no-interaction --no-progress --optimize-autoloader
if [ $? -eq 0 ]; then if [ $? -eq 0 ]; then
echo "Successfully installed additional composer packages" echo "Successfully installed additional composer packages"

View file

@ -42,6 +42,9 @@ fi
# Install additional composer packages if COMPOSER_EXTRA_PACKAGES is set # Install additional composer packages if COMPOSER_EXTRA_PACKAGES is set
if [ -n "$COMPOSER_EXTRA_PACKAGES" ]; then if [ -n "$COMPOSER_EXTRA_PACKAGES" ]; then
echo "Installing additional composer packages: $COMPOSER_EXTRA_PACKAGES" 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 sudo -E -u www-data composer require $COMPOSER_EXTRA_PACKAGES --no-interaction --no-progress --optimize-autoloader
if [ $? -eq 0 ]; then if [ $? -eq 0 ]; then
echo "Successfully installed additional composer packages" echo "Successfully installed additional composer packages"