- Issue created by @longwave
- ππΊHungary GΓ‘bor Hojtsy Hungary
There is also π [META] Reduce use of ContainerAware classes where possible Active , how are the two related? :)
- π¬π§United Kingdom longwave UK
Reparenting this under the "reduce" meta, which when reduced to zero can be closed as fixed.
- Status changed to Active
8 months ago 9:59am 20 March 2024 - π¬π§United Kingdom longwave UK
In fact this doesn't need to be postponed, we can remove the remaining support except for LoggerChannelFactory in this issue now.
- Assigned to spokje
- Status changed to Needs review
8 months ago 11:59am 20 March 2024 - π³π±Netherlands spokje
Five references left for
ContainerAwareInterface
after this MR.They're all needed to keep
LoggerChannelFactory
working for now. - Status changed to Needs work
8 months ago 12:21pm 20 March 2024 - π¬π§United Kingdom longwave UK
One case where we lost test coverage and can replace it with something else, but otherwise this looks good.
- Status changed to Needs review
8 months ago 12:48pm 20 March 2024 - Issue was unassigned.
- Status changed to RTBC
8 months ago 3:09pm 20 March 2024 - Status changed to Fixed
8 months ago 8:43am 21 March 2024 -
alexpott β
committed a058be43 on 11.x
Issue #3431362 by Spokje, longwave: Remove support for...
-
alexpott β
committed a058be43 on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.