Skip to content

Commit

Permalink
Merge pull request #23 from phpro/PHP83
Browse files Browse the repository at this point in the history
Allow PHP83
  • Loading branch information
veewee authored Nov 24, 2023
2 parents 466aabf + 434f720 commit 51872e9
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 13 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/grumphp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ jobs:
strategy:
matrix:
operating-system: [ubuntu-latest]
php-versions: ['8.0', '8.1', '8.2']
php-versions: ['8.1', '8.2', '8.3']
composer-options: ['', '--prefer-lowest']
fail-fast: false
name: PHP ${{ matrix.php-versions }} @ ${{ matrix.operating-system }} with ${{ matrix.composer-options }}
Expand Down Expand Up @@ -39,4 +39,4 @@ jobs:
- name: Run the tests
run: php vendor/bin/grumphp run --no-interaction
env:
PHP_CS_FIXER_IGNORE_ENV: 1
PHP_CS_FIXER_IGNORE_ENV: 1
16 changes: 8 additions & 8 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,20 +10,20 @@
}
],
"require": {
"php": "^8.0",
"php": "~8.1.0 || ~8.2.0 || ~8.3.0",
"phpro/api-problem": "^1.0",
"symfony/dependency-injection": "^5.4 || ^6.0",
"symfony/event-dispatcher": "^5.4 || ^6.0",
"symfony/http-kernel": "^5.4 || ^6.0"
"symfony/dependency-injection": "^5.4 || ^6.0 || ^7.0",
"symfony/event-dispatcher": "^5.4 || ^6.0 || ^7.0",
"symfony/http-kernel": "^5.4 || ^6.0 || ^7.0"
},
"require-dev": {
"friendsofphp/php-cs-fixer": "^3.13",
"friendsofphp/php-cs-fixer": "^3.39",
"matthiasnoback/symfony-dependency-injection-test": "^4.3",
"phpro/grumphp-shim": "^1.14.0",
"phpro/grumphp-shim": "^2.3",
"phpspec/prophecy-phpunit": "^2.0",
"phpspec/prophecy": "1.15",
"phpspec/prophecy": "^1.17",
"phpunit/phpunit": "^9.5",
"symfony/security-core": "^5.4 || ^6.0"
"symfony/security-core": "^5.4 || ^6.0 || ^7.0"
},
"config": {
"sort-packages": true,
Expand Down
4 changes: 2 additions & 2 deletions src/EventListener/JsonApiProblemExceptionListener.php
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,8 @@ public function onKernelException(ExceptionEvent $event): void
{
$request = $event->getRequest();
if (
false === mb_strpos($request->getPreferredFormat(), 'json') &&
false === mb_strpos((string) $request->getContentType(), 'json')
false === mb_strpos($request->getPreferredFormat(), 'json')
&& false === mb_strpos((string) $request->getContentType(), 'json')
) {
return;
}
Expand Down
2 changes: 1 addition & 1 deletion test/Transformer/ChainTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ public function it_transforms_to_basic_exception_problem_when_no_transformer_mat
$this->assertInstanceOf(ExceptionApiProblem::class, $transformer->transform(new Exception()));
}

private function mockTransformer(bool $accepts, ?ApiProblemInterface $apiProblem = null): ExceptionTransformerInterface
private function mockTransformer(bool $accepts, ApiProblemInterface $apiProblem = null): ExceptionTransformerInterface
{
/** @var ExceptionTransformerInterface|ObjectProphecy $transformer */
$transformer = $this->prophesize(ExceptionTransformerInterface::class);
Expand Down

0 comments on commit 51872e9

Please sign in to comment.