core/php-developer
8
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(ci): correct bash escaping in dependency checkout step
Some checks failed
CI / PHP 8.4 (push) Waiting to run
Details
CI / PHP 8.3 (push) Has been cancelled
Details

Browse source 
The PHP variables inside php -r need \$ escaping, but shell
variables outside need bare $ for command substitution and
variable expansion.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Claude 2026-02-23 05:19:16 +00:00
parent 8160ecec00
commit 8f3fb2c9c8
No known key found for this signature in database
GPG key ID: AF404715446AEB41
1 changed files with 15 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
.forgejo/workflows/ci.yml
View file

@ -26,11 +26,10 @@ jobs:
extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite
coverage: pcov coverage: pcov
- name: Checkout dependencies - name: Checkout path dependencies
env: env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: | run: |
if grep -q '"path":' composer.json 2>/dev/null; then
for path in $(php -r " for path in $(php -r "
\$d = json_decode(file_get_contents('composer.json'), true); \$d = json_decode(file_get_contents('composer.json'), true);
foreach (\$d['repositories'] ?? [] as \$r) { foreach (\$d['repositories'] ?? [] as \$r) {
@ -45,7 +44,6 @@ jobs:
"$path" || echo "Warning: Failed to clone $dir_name" "$path" || echo "Warning: Failed to clone $dir_name"
fi fi
done done
fi
- name: Install dependencies - name: Install dependencies
run: composer install --prefer-dist --no-interaction --no-progress run: composer install --prefer-dist --no-interaction --no-progress