Skip to content

Commit 6f5c02d

Browse files
committed
minor #57246 [HttpKernel] Remove @internal flag and add @final to ServicesResetter (lyrixx)
This PR was merged into the 7.2 branch. Discussion ---------- [HttpKernel] Remove ``@internal`` flag and add ``@final`` to `ServicesResetter` | Q | A | ------------- | --- | Branch? | 7.2 | Bug fix? | no | New feature? | yes | Deprecations? | no | Issues | | License | MIT I used this class for ages! I moved many of my internal "hack" to symfony (messenger, etc) But there are still a lot of use cases where we could need this class: * Custom worker that does not use messenger * import/export command that process a lot of rows (doctrine/dbal leak by default, even in prod env) * etc --- Anyway, I used this class, and I never noticed it was internal. And today, I showed this class to `@pyrech` and its IDE displayed the class in strikethrough! (vscode does not do that...) So I think it's time to open this class. Commits ------- 6b1032f59b Remove ``@internal`` flag and add ``@final`` to `ServicesResetter`
2 parents c4b2ae5 + 4d599f9 commit 6f5c02d

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
CHANGELOG
22
=========
33

4+
7.2
5+
---
6+
7+
* Remove `@internal` flag and add `@final` to `ServicesResetter`
8+
49
7.1
510
---
611

DependencyInjection/ServicesResetter.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
* @author Alexander M. Turek <me@derrabus.de>
2222
* @author Nicolas Grekas <p@tchwork.com>
2323
*
24-
* @internal
24+
* @final since Symfony 7.2
2525
*/
2626
class ServicesResetter implements ResetInterface
2727
{

0 commit comments

Comments
 (0)