Skip to content

Commit

Permalink
PHP 8.4 compatibility (#222)
Browse files Browse the repository at this point in the history
* Add PHP 8.3 and PHP 8.4 to matrix

* Make compatible with PHP 8.4

> MessageBird\Client::__construct(): Implicitly marking parameter $httpClient as nullable is deprecated, the explicit nullable type must be used instead
  • Loading branch information
ruudk authored Nov 1, 2024
1 parent 1509430 commit 47e7652
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
strategy:
fail-fast: true
matrix:
php: [ '7.4', '8.0', '8.1', '8.2' ]
php: [ '7.4', '8.0', '8.1', '8.2', '8.3', '8.4' ]
stability: [ prefer-stable ]

name: PHP ${{ matrix.php }} - ${{ matrix.stability }}
Expand Down
2 changes: 1 addition & 1 deletion src/MessageBird/Client.php
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ class Client
*/
protected $numbersAPIClient;

public function __construct(?string $accessKey = null, Common\HttpClient $httpClient = null, array $config = [])
public function __construct(?string $accessKey = null, ?Common\HttpClient $httpClient = null, array $config = [])
{
if ($httpClient === null) {
$this->conversationsAPIHttpClient = new Common\HttpClient(
Expand Down

0 comments on commit 47e7652

Please sign in to comment.