Skip to content

Commit

Permalink
Merge pull request #543 from FriendsOfSymfony/fix-php-http-dependency
Browse files Browse the repository at this point in the history
php-http/message stopped providing php-http/message-factory
  • Loading branch information
dbu committed May 17, 2023
2 parents 652f4f5 + 1fe8f1c commit 0f3b819
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ jobs:

legacyPHP:
name: PHP 7.2 Varnish 6
runs-on: ubuntu-18.04
runs-on: ubuntu-20.04
env:
VARNISH_VERSION: '6.6'
VARNISH_MODULES_VERSION: '0.18.0'
Expand Down Expand Up @@ -169,7 +169,7 @@ jobs:

lowest:
name: PHP ${{ matrix.php }} Lowest, Varnish 3
runs-on: ubuntu-18.04
runs-on: ubuntu-20.04
env:
VARNISH_VERSION: '3.0'
VARNISH_MODULES_VERSION: ''
Expand Down
1 change: 1 addition & 0 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
"php-http/client-implementation": "^1.0 || ^2.0",
"php-http/client-common": "^1.1.0 || ^2.0",
"php-http/message": "^1.0 || ^2.0",
"php-http/message-factory": "^1.0",
"php-http/discovery": "^1.12"
},
"require-dev": {
Expand Down
4 changes: 3 additions & 1 deletion tests/Functional/Varnish/UserContextFailureTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,9 @@ class UserContextFailureTest extends VarnishTestCase
public function setUp(): void
{
// needs to be decided before doing the setup
$this->mode = 'testHashRequestFailure' === $this->getName() ? 'failure' : 'cache';
// phpunit 9 calls the method getName(), phpunit 10 name()
$name = method_exists($this, 'name') ? $this->name() : $this->getName(); /* @phpstan-ignore-line */
$this->mode = 'testHashRequestFailure' === $name ? 'failure' : 'cache';

parent::setUp();
}
Expand Down

0 comments on commit 0f3b819

Please sign in to comment.