Skip to content

Commit

Permalink
Update to symfony7 (#63)
Browse files Browse the repository at this point in the history
* Update to symfony7

* Update testing strategy for Symfony 7

* Remove support for PHP verions < 8.2

* Update lockfile

---------

Co-authored-by: Lee Rowlands <lee.rowlands@previousnext.com.au>
Co-authored-by: Greg Anderson <greg-1-andreson@greenknowe.org>
  • Loading branch information
3 people authored Apr 5, 2024
1 parent be02163 commit ab38640
Show file tree
Hide file tree
Showing 4 changed files with 268 additions and 344 deletions.
28 changes: 7 additions & 21 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
- name: Install PHP
uses: shivammathur/setup-php@v2
with:
php-version: 8.1
php-version: 8.2
coverage: none

- name: Install dependencies
Expand Down Expand Up @@ -53,7 +53,7 @@ jobs:
- name: Install PHP with extensions
uses: shivammathur/setup-php@v2
with:
php-version: 8.1
php-version: 8.2
coverage: none
extensions: intl

Expand Down Expand Up @@ -85,34 +85,20 @@ jobs:
- windows-latest

php-version:
- "8.1"
- "8.2"

php-ini-values:
- assert.exception=1, zend.assertions=1

dependencies:
- locked

codecov:
- false

include:
- os: ubuntu-latest
php-version: "7.1"
dependencies: lowest

- os: ubuntu-latest
php-version: "7.4"
dependencies: lowest

- os: ubuntu-latest
php-version: "8.0"
dependencies: highest
codecov: true
php-ini-values: assert.exception=1, zend.assertions=1, opcache.enable=1, opcache.enable_cli=1, opcache.optimization_level=-1, opcache.jit_buffer_size=4096M, opcache.jit=1205

- os: ubuntu-latest
php-version: "8.2"
php-version: "8.3"
dependencies: highest
codecov: true
php-ini-values: assert.exception=1, zend.assertions=1, opcache.enable=1, opcache.enable_cli=1, opcache.optimization_level=-1, opcache.jit_buffer_size=4096M, opcache.jit=1205
Expand Down Expand Up @@ -147,7 +133,7 @@ jobs:
- name: Make sure composer.json is valid before we start modifyig it
run: composer validate

- name: Clear platform php configuration in case we need to update phpunit
run: composer config --unset platform.php

Expand All @@ -167,7 +153,7 @@ jobs:

- name: Run tests with phpunit
run: composer unit

- name: Publish code coverage to Codecov
if: matrix.codecov == true
run: bash <(curl -s https://codecov.io/bash)
16 changes: 8 additions & 8 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,17 +19,17 @@
}
},
"require": {
"php": ">=7.1.3",
"dflydev/dot-access-data": "^1.1.0 || ^2 || ^3",
"grasmash/expander": "^2.0.1 || ^3",
"symfony/event-dispatcher": "^4 || ^5 || ^6"
"php": ">=8.2.0",
"dflydev/dot-access-data": "^3",
"grasmash/expander": "^3",
"symfony/event-dispatcher": "^7"
},
"require-dev": {
"ext-json": "*",
"phpunit/phpunit": "^7.5.20 || ^8 || ^9",
"phpunit/phpunit": "^9",
"squizlabs/php_codesniffer": "^3",
"symfony/console": "^4 || ^5 || ^6",
"symfony/yaml": "^4 || ^5 || ^6",
"symfony/console": "^7",
"symfony/yaml": "^7",
"yoast/phpunit-polyfills": "^1"
},
"suggest": {
Expand All @@ -40,7 +40,7 @@
"optimize-autoloader": true,
"sort-packages": true,
"platform": {
"php": "8.1.14"
"php": "8.2.17"
}
},
"scripts": {
Expand Down
Loading

0 comments on commit ab38640

Please sign in to comment.