Skip to content

Commit

Permalink
Merge pull request #309 from akrabat/php83
Browse files Browse the repository at this point in the history
Ensure that we work with PHP 8.3
  • Loading branch information
akrabat committed Jun 8, 2024
2 parents f583b1c + d7b6767 commit 372e11c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
strategy:
fail-fast: false
matrix:
php: [8.0, 8.1, 8.2]
php: [8.0, 8.1, 8.2, 8.3]
experimental: [false]
include:
- php: 8.2
Expand Down
3 changes: 2 additions & 1 deletion phpstan.neon.dist
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
parameters:
level: max
checkMissingIterableValueType: false
ignoreErrors:
- identifier: missingType.iterableValue
paths:
- src
2 changes: 1 addition & 1 deletion src/Stream.php
Original file line number Diff line number Diff line change
Expand Up @@ -304,7 +304,7 @@ public function read($length): string
{
$data = false;

if ($this->isReadable() && $this->stream && $length >= 0) {
if ($this->isReadable() && $this->stream && $length > 0) {
$data = fread($this->stream, $length);
}

Expand Down

0 comments on commit 372e11c

Please sign in to comment.