Skip to content
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

Add CI job for PHP 8.4 #11631

Merged
merged 2 commits into from
Oct 9, 2024
Merged

Add CI job for PHP 8.4 #11631

merged 2 commits into from
Oct 9, 2024

Conversation

greg0ire
Copy link
Member

@greg0ire greg0ire commented Oct 8, 2024

@greg0ire greg0ire marked this pull request as draft October 8, 2024 15:04
@greg0ire greg0ire marked this pull request as ready for review October 8, 2024 15:25
@beberlei beberlei mentioned this pull request Oct 8, 2024
6 tasks
@greg0ire greg0ire requested review from beberlei and derrabus October 8, 2024 15:48
For now doctrine/common generates proxies that trigger deprecation, so
let us only test with lazy ghosts only.
SenseException
SenseException previously approved these changes Oct 8, 2024
@SenseException
Copy link
Member

Nice, that it seems to work with PHP 8.4 out of the box.

tests/Tests/TestInit.php Outdated Show resolved Hide resolved
@derrabus derrabus added the CI label Oct 8, 2024
@derrabus
Copy link
Member

derrabus commented Oct 8, 2024

Shall we target 2.19?

derrabus
derrabus previously approved these changes Oct 8, 2024
@beberlei
Copy link
Member

beberlei commented Oct 8, 2024

We need to release 2.20 with a more undeprecations in the next few days anyways, so 2.20 for me is fine.

@greg0ire greg0ire dismissed stale reviews from derrabus and SenseException via 8375027 October 9, 2024 06:44
E_STRICT is deprecated as of PHP 8.4
@greg0ire greg0ire merged commit 109042e into doctrine:2.20.x Oct 9, 2024
71 checks passed
@greg0ire greg0ire deleted the php84-ci branch October 9, 2024 07:42
@greg0ire greg0ire added this to the 2.20.0 milestone Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants