Skip to content

Commit 0912c55

Browse files
Florian Guimierclaude
andcommitted
Stop committing composer.lock, resolve dependencies per CI matrix entry
The lock was resolved on PHP 8.4 (symfony/serializer 7.4 requires PHP >= 8.2), which broke 'composer install' on the PHP 8.1 matrix job. Libraries should not pin a lock: the phpunit workflow now runs 'composer update' so each matrix PHP version resolves its own compatible dependency set (8.1 -> serializer 6.4, 8.4 -> serializer 8.x). Cache key moved to composer.json accordingly. Note: the 'PHP Linting' workflow failure is pre-existing (since Jan 2026): .php-cs-fixer.php-highest.php aborts unless it runs on PHP 8.1 while the php-cs-fixer-ga docker image has moved to a newer PHP. Left for a dedicated maintenance pass. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 5e31338 commit 0912c55

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/phpunit.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,11 @@ jobs:
2121
uses: actions/cache@v3
2222
with:
2323
path: vendor
24-
key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
24+
key: ${{ runner.os }}-php-${{ hashFiles('**/composer.json') }}
2525
restore-keys: |
2626
${{ runner.os }}-php-
2727
- name: Install Dependencies
2828
run: |
29-
composer install
29+
composer update --no-interaction
3030
- name: Execute PHPUnit tests
3131
run: vendor/bin/phpunit

.gitignore

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@
33
.php_cs.cache
44
.php-cs-fixer.cache
55
.phpunit.result.cache
6-
test.php
6+
test.phpcomposer.lock

0 commit comments

Comments
 (0)