Allow Symfony 8

[ajgarlag]

This will allow to run the test suite against Symfony 8. Once it's merged, I'll open a new PR to revert this one that should be merged before releasing a new minor version. See doctrine/orm#12110

Fix #1910

[ajgarlag]

If we allow symfony/console:^8.0 or symfony/dependency-injection:^8.0 the phpunit execution fails with:

Run vendor/bin/phpunit --coverage-clover=coverage.xml
PHPUnit 12.4-dev by Sebastian Bergmann and contributors.

Runtime:       PHP 8.4.12 with PCOV 1.0.12
Configuration: /home/runner/work/DoctrineBundle/DoctrineBundle/phpunit.xml.dist

Fatal error: Premature end of PHP process when running Doctrine\Bundle\DoctrineBundle\Tests\BundleTest::testBuildCompilerPasses.
Error: Process completed with exit code 255.

See test

[dmaicher]

If we allow symfony/console:^8.0 or symfony/dependency-injection:^8.0 the phpunit execution fails with:

Yeah it fails due to

Fatal error: Declaration of Doctrine\Bundle\DoctrineBundle\Command\DropDatabaseDoctrineCommand::configure() must be compatible with Symfony\Component\Console\Command\Command::configure(): void in /var/www/DoctrineBundle/src/Command/DropDatabaseDoctrineCommand.php on line 33

and

Fatal error: Declaration of Doctrine\Bundle\DoctrineBundle\DependencyInjection\Compiler\EntityListenerPass::process(Symfony\Component\DependencyInjection\ContainerBuilder $container) must be compatible with Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface::process(Symfony\Component\DependencyInjection\ContainerBuilder $container): void in /var/www/DoctrineBundle/src/DependencyInjection/Compiler/EntityListenerPass.php on line 31

Fatal error: Declaration of Doctrine\Bundle\DoctrineBundle\DependencyInjection\Compiler\DbalSchemaFilterPass::process(Symfony\Component\DependencyInjection\ContainerBuilder $container) must be compatible with Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface::process(Symfony\Component\DependencyInjection\ContainerBuilder $container): void in /var/www/DoctrineBundle/src/DependencyInjection/Compiler/DbalSchemaFilterPass.php on line 22

Fatal error: Declaration of Doctrine\Bundle\DoctrineBundle\DependencyInjection\Compiler\CacheSchemaSubscriberPass::process(Symfony\Component\DependencyInjection\ContainerBuilder $container) must be compatible with Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface::process(Symfony\Component\DependencyInjection\ContainerBuilder $container): void in /var/www/DoctrineBundle/src/DependencyInjection/Compiler/CacheSchemaSubscriberPass.php on line 22

Fatal error: Declaration of Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface@anonymous::process(Symfony\Component\DependencyInjection\ContainerBuilder $container) must be compatible with Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface::process(Symfony\Component\DependencyInjection\ContainerBuilder $container): void in /var/www/DoctrineBundle/tests/DependencyInjection/Compiler/RemoveLoggingMiddlewarePassTest.php on line 49

It might be ok to update those signatures to add the native void return type? 🤔 all the classes are marked @final since a while already.

[ajgarlag]

@dmaicher The Fatal error: Premature end of PHP process message with symfony/console:^8 and symfony/dependency-injection:^8 is not related to the void return type.

See: ajgarlag#2

[dmaicher]

@dmaicher The Fatal error: Premature end of PHP process message with symfony/console:^8 and symfony/dependency-injection:^8 is not related to the void return type.

See: ajgarlag#2

Hm interesting. I tried it locally and I did not have any other error though 🤔