You are right, I picked the wrong one 😅

Sorry, I'm late in the game here, but what is the rationale behind doing a BC break in 7.1?

Sorry, I'm late in the game here, but what is the rationale behind doing a BC break in 7.1?

Hey Fabien, we published this as a BC break because technically it is (we changed the behavior of theimport() method under certain context/scope without deprecation path). However, actually, nobody can/should use theimport() method in theprependExtension() because it's defective, appending config instead of prepending. That's why we were using theContainerBuilder::prependExtensionConfig() until now (which is limited to anarray of config).

Honestly, this BC break, in the end, doesn't feel like a real BC break to me, as the previous behavior was faulty and unwanted. However, it applies to the BC break rules, so we are documenting it as such.

More details can be found here#52789 and here#52843

Hope it's ok for you🤞

Sorry, I'm late in the game here, but what is the rationale behind doing a BC break in 7.1?

Hey Fabien, we published this as a BC break because technically it is (we changed the behavior of theimport() method under certain context/scope without deprecation path). However, actually, nobody can/should use theimport() method in theprependExtension() because it's defective, appending config instead of prepending. That's why we were using theContainerBuilder::prependExtensionConfig() until now (which is limited to anarray of config).

Honestly, this BC break, in the end, doesn't feel like a real BC break to me, as the previous behavior was faulty and unwanted. However, it applies to the BC break rules, so we are documenting it as such.

More details can be found here#52789 and here#52843

Hope it's ok for you🤞

Thank you for the clear explanation, that makes sense to me now.

Thank you@OskarStark.