Skip to content

Run tests on PHP 8.2#5442

Merged
derrabus merged 1 commit intodoctrine:3.3.xfrom
derrabus:ci/php-8.2
Jun 23, 2022
Merged

Run tests on PHP 8.2#5442
derrabus merged 1 commit intodoctrine:3.3.xfrom
derrabus:ci/php-8.2

Conversation

@derrabus
Copy link
Member

PHP 8.2.0-alpha1 has been released and I think we should start monitoring our compatibility with the next PHP version.

@derrabus
Copy link
Member Author

The tests fail with OCI8 on this line:

Creation of dynamic property OCILob::$descriptor is deprecated

$lob = oci_new_descriptor($this->connection, OCI_D_LOB);

Looks like a PHP bug to me.

@derrabus
Copy link
Member Author

Reported as php/php-src#8756

@derrabus derrabus marked this pull request as ready for review June 23, 2022 21:58
@derrabus
Copy link
Member Author

The OCI8 issue has been fixed upstream. CI is green now.

@derrabus derrabus requested a review from morozov June 23, 2022 21:59
@morozov
Copy link
Member

morozov commented Jun 23, 2022

The OCI8 issue has been fixed upstream. CI is green now.

Thanks for taking care of it, @derrabus!

@morozov morozov added this to the 3.3.8 milestone Jun 23, 2022
@derrabus derrabus merged commit b178d60 into doctrine:3.3.x Jun 23, 2022
@derrabus derrabus deleted the ci/php-8.2 branch June 23, 2022 22:17
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants