Skip to content

Conversation

Lanatel
Copy link

@Lanatel Lanatel commented Jul 29, 2025

This pull request adds support for PHP 8.4 by updating the CI workflow and ensuring compatibility.

Deprecation notices were appearing under PHP 8.4, such as:

Deprecated: LaravelDoctrine\Fluent\Fluent::hasMany(): Implicitly marking parameter $callback as nullable is deprecated

@Lanatel Lanatel force-pushed the add-php-8.4-support branch from 7c190d9 to 81d9e88 Compare July 29, 2025 14:28
@TomHAnderson TomHAnderson merged commit 221aa35 into laravel-doctrine:1.x Jul 29, 2025
1 check passed
@Lanatel
Copy link
Author

Lanatel commented Jul 30, 2025

Hi @TomHAnderson , would you be able to create a new tag when you have a moment?

Also, I noticed a StyleCI failure after the merge - it seems unrelated to my changes (the flagged file wasn't modified in the PR), but I'm happy to open a follow-up PR to fix it if needed.

Thanks!

@TomHAnderson
Copy link
Contributor

I waited on the tag because of the CI failure. I would appreciate it if you'd look into that. This library is an odd duck. But as long as folks like yourself are using it, I'll continue to maintain it.

@Lanatel
Copy link
Author

Lanatel commented Jul 31, 2025

@TomHAnderson I’ve opened a new PR with the fixes: #84. Let me know if anything else is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants