Skip to content

Conversation

@HypeMC
Copy link
Member

@HypeMC HypeMC commented Sep 27, 2025

Q A
Branch? 7.4
Bug fix? no
New feature? no
Deprecations? no
Issues -
License MIT

Follow-up to #60645

The SymfonyExtension::bootstrap() method seems like a more appropriate place to configure Doctrine deprecations behavior.

With this, once support for PHPUnit <10 i dropped, the bootstrap.php file can be completely removed.

@nicolas-grekas
Copy link
Member

Thank you @HypeMC.

@nicolas-grekas nicolas-grekas merged commit 145fa98 into symfony:7.4 Sep 29, 2025
12 of 13 checks passed
@HypeMC HypeMC deleted the doctrine-deprecations branch September 29, 2025 17:25
alexandre-daubois added a commit that referenced this pull request Oct 29, 2025
This PR was merged into the 7.4 branch.

Discussion
----------

 Fix the initialization of the phpunit bridge

| Q             | A
| ------------- | ---
| Branch?       | 7.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        |
| License       | MIT

When PHPUnit is installed through composer, the bootstrap.php file will be loaded *before* the `Metadata` class is autoloaded by PHPUnit itself. This was breaking the skipping of the initialization, which broke deprecation reporting by mixing the old system and PHPUnit 11 in my case.
Before #61863, the old checks were triggering autoloading.

Commits
-------

37c13ef Fix the initialization of the phpunit bridge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants