Skip to content

Commit

Permalink
Test 8.2 and 8.3 PHP against Symfony 7 rest against stable branches
Browse files Browse the repository at this point in the history
  • Loading branch information
alexander-schranz committed Jun 10, 2024
1 parent 059da68 commit b3989ef
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/test-application.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,10 @@ jobs:
- name: Remove not required test tooling
run: composer remove "*php-cs-fixer*" "*phpstan*" --no-update --dev

- name: Install Symfony 7 dev branch # TODO remove
if: ${{ matrix.php-version == '8.2' || matrix.php-version == '8.3' }}
run: composer require sulu/sulu:"dev-feature/symfony-7-support as 2.6.3" --no-update

- name: Install composer dependencies
uses: ramsey/composer-install@v3
with:
Expand Down
3 changes: 2 additions & 1 deletion Tests/Functional/Controller/NavigationControllerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,8 @@ public function provideAttributes(): \Generator
$suluSuluVersion = \Composer\InstalledVersions::getVersion('sulu/sulu');

/* Todo Can be removed after ending support of sulu 2.5 */
if (\version_compare($suluSuluVersion ?? '999.999.999', '2.6.0', '<')
if ($suluSuluVersion
&& \version_compare($suluSuluVersion ?? '999.999.999', '2.6.0', '<')

Check failure on line 121 in Tests/Functional/Controller/NavigationControllerTest.php

View workflow job for this annotation

GitHub Actions / PHP Lint

Variable $suluSuluVersion on left side of ?? always exists and is not nullable.
&& 0 !== \strpos($suluSuluVersion, 'dev-')
) {
$path = '/legacy/';
Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
],
"require": {
"php": "^7.3 || ^8.0",
"sulu/sulu": "dev-feature/symfony-7-support as 2.6.3",
"sulu/sulu": "^2.4 || ^2.5@dev",
"symfony/config": "^4.4 || ^5.0 || ^6.0 || ^7.0",
"symfony/dependency-injection": "^4.4 || ^5.0 || ^6.0 || ^7.0",
"symfony/framework-bundle": "^4.4 || ^5.0 || ^6.0 || ^7.0",
Expand Down

0 comments on commit b3989ef

Please sign in to comment.