New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run tests on PHP 8.1 with different database drivers #4875
Conversation
Marking as draft because I expect tests to fail. |
9520777
to
443866b
Compare
@@ -434,7 +433,7 @@ jobs: | |||
coverage: "pcov" | |||
ini-values: "zend.assertions=1" | |||
tools: "pecl" | |||
extensions: "${{ matrix.extension }}-5.9.0preview1" | |||
extensions: "${{ matrix.extension }}-5.10.0beta1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Earlier releases of the driver are not compatible with PHP 8.1.
443866b
to
840caef
Compare
840caef
to
5094ab3
Compare
Tests are green. As usual, we need to adjust the branch protection rules. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|
||
public static function upcast(mysqli_sql_exception $exception): self | ||
{ | ||
$p = new ReflectionProperty(mysqli_sql_exception::class, 'sqlstate'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering, what the idea behind the mysqli_sql_exception
API design is 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah. A protected property without any accessors is a really brilliant idea.
I've bumped all PHP 8.0 jobs for individual database drivers to PHP 8.1 to uncover issue that we might have there. If you think that testing on PHP 8.0 additionally provides extra value, I can re-add those jobs again. But for now, I did not want to bloat the test matrix further.
Fixes #4870