Skip to content

Commit 09a8860

Browse files
authored
Merge pull request #25 from greg-1-anderson/symfony-7-part-2
Update tests to run Symfony 7, and update function signatures to match Symfony's requirements.
2 parents a5af7c1 + 1946eba commit 09a8860

File tree

9 files changed

+138
-223
lines changed

9 files changed

+138
-223
lines changed

.github/workflows/php.yml

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,15 @@ jobs:
1818
include:
1919
- os: "ubuntu-latest"
2020
php: "8.0"
21-
env: 'HIGHEST_LOWEST="update --prefer-lowest"'
2221
coverage: "none"
2322
- os: "ubuntu-latest"
2423
php: "8.1"
25-
env: 'HIGHEST_LOWEST="update"'
24+
coverage: "none"
25+
- os: "ubuntu-latest"
26+
php: "8.2"
27+
coverage: "pcov"
28+
- os: "ubuntu-latest"
29+
php: "8.3"
2630
coverage: "pcov"
2731

2832
steps:
@@ -37,17 +41,11 @@ jobs:
3741
- name: Validate composer.json and composer.lock
3842
run: composer validate --strict
3943

40-
- name: Cache Composer packages
41-
id: composer-cache
42-
uses: actions/cache@v3
43-
with:
44-
path: vendor
45-
key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
46-
restore-keys: |
47-
${{ runner.os }}-php-
44+
- name: Un-pin PHP version used for dependency resolution
45+
run: composer config --unset platform.php
4846

49-
- name: Install dependencies
50-
run: composer -n ${HIGHEST_LOWEST-install} --prefer-dist -o
47+
- name: Install dependencies; use "update" so that we can get the right dependencies for our php version
48+
run: composer -n update --prefer-dist -o
5149

5250
# Add a test script to composer.json, for instance: "test": "vendor/bin/phpunit"
5351
# Docs: https://getcomposer.org/doc/articles/scripts.md

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@
6060
},
6161
"config": {
6262
"platform": {
63-
"php": "8.0.2"
63+
"php": "8.2.18"
6464
}
6565
}
6666
}

0 commit comments

Comments
 (0)