Fix implicit nullables for PHP 8.4

This applies to 3.x branch too.

Ah there were a couple missing

liam morland → made their first commit to this issue’s fork.

All tests are now passing except cspell and eslint.

Most issues have been fixed in 🐛 Fix PHP 8.4.x deprecation and other warnings (PHPStan) Active . The remaining changes should still be committed.

Should we block this issue on 🐛 Fix PHP 8.4.x deprecation and other warnings (PHPStan) Active that I think should be committed first?

The order doesn't much matter. The commit message for this one need to be changed, though. The nullable types issue has already been fixed.

As of #11 - this is close to the finish line :)

The original scope here was fixing implicit nullable deprecations which has already been done.

All that is left in the merge request is adding a return type declaration.

Nice, let's also solve that!

Great, thank you @liam morland!

Automatically closed - issue fixed for 2 weeks with no activity.