Skip to content

Commit 354d201

Browse files
Copilotjbtronics
andcommitted
Add shellcheck disable comment for intentional word splitting
Co-authored-by: jbtronics <[email protected]>
1 parent 4399616 commit 354d201

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

.docker/frankenphp/docker-entrypoint.sh

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,9 @@ if [ "$1" = 'frankenphp' ] || [ "$1" = 'php' ] || [ "$1" = 'bin/console' ]; then
2929
# Install additional composer packages if COMPOSER_EXTRA_PACKAGES is set
3030
if [ -n "$COMPOSER_EXTRA_PACKAGES" ]; then
3131
echo "Installing additional composer packages: $COMPOSER_EXTRA_PACKAGES"
32+
# Note: COMPOSER_EXTRA_PACKAGES is intentionally not quoted to allow word splitting
33+
# This enables passing multiple package names separated by spaces
34+
# shellcheck disable=SC2086
3235
composer require $COMPOSER_EXTRA_PACKAGES --no-interaction --no-progress --optimize-autoloader
3336
if [ $? -eq 0 ]; then
3437
echo "Successfully installed additional composer packages"

.docker/partdb-entrypoint.sh

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,9 @@ fi
4242
# Install additional composer packages if COMPOSER_EXTRA_PACKAGES is set
4343
if [ -n "$COMPOSER_EXTRA_PACKAGES" ]; then
4444
echo "Installing additional composer packages: $COMPOSER_EXTRA_PACKAGES"
45+
# Note: COMPOSER_EXTRA_PACKAGES is intentionally not quoted to allow word splitting
46+
# This enables passing multiple package names separated by spaces
47+
# shellcheck disable=SC2086
4548
sudo -E -u www-data composer require $COMPOSER_EXTRA_PACKAGES --no-interaction --no-progress --optimize-autoloader
4649
if [ $? -eq 0 ]; then
4750
echo "Successfully installed additional composer packages"

0 commit comments

Comments
 (0)