Skip to content

Commit

Permalink
Test with guzzle/PSR7 2.0 (#2874)
Browse files Browse the repository at this point in the history
* Test with guzzle/PSR7 2.0

* typo

* bugfix

* Allow dev packages

* Adding return type hints

* Test with the bleeding edge of guzzlehttp/psr7

* Test with stable/beta and dev.

* Test with dev-master for 1.x too

* Reorder.

* Dont test on beta
  • Loading branch information
Nyholm committed Mar 21, 2021
1 parent 0103c78 commit f769147
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
10 changes: 9 additions & 1 deletion .github/workflows/ci.yml
Expand Up @@ -43,6 +43,12 @@ jobs:
max-parallel: 10
matrix:
php: ['7.2', '7.3', '7.4', '8.0']
psr7: ['^1.7']
include:
- php: '7.4'
psr7: '^1.7@dev'
- php: '7.4'
psr7: '^2.0@dev'

steps:
- name: Set up PHP
Expand All @@ -64,7 +70,9 @@ jobs:
uses: actions/checkout@v2

- name: Download dependencies
run: composer update --no-interaction --no-progress --prefer-dist
run: |
composer require --no-update "guzzlehttp/psr7:${{ matrix.psr7 }}"
composer update --no-interaction --no-progress --prefer-dist
- name: Run tests
run: make test
Expand Down
4 changes: 2 additions & 2 deletions tests/Exception/RequestExceptionTest.php
Expand Up @@ -184,12 +184,12 @@ public function __construct()
parent::__construct(\fopen('php://memory', 'wb'));
}

public function isSeekable()
public function isSeekable(): bool
{
return true;
}

public function isReadable()
public function isReadable(): bool
{
return false;
}
Expand Down

0 comments on commit f769147

Please sign in to comment.