You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Some time ago injection of Container/Config (and some other) were replaced by the direct call Container::getInstance() to be compatible with Octane (#38). The main problem with the current approach - implicit dependencies and (more important) implicit config (Container::getInstance()->make(Repository::class)->get(...)).
Recently I have reviewed all these places and looks like it is possible to convert all of them into explicit 🎉
Some time ago injection of
Container
/Config
(and some other) were replaced by the direct callContainer::getInstance()
to be compatible with Octane (#38). The main problem with the current approach - implicit dependencies and (more important) implicit config (Container::getInstance()->make(Repository::class)->get(...)
).Recently I have reviewed all these places and looks like it is possible to convert all of them into explicit 🎉
Container::getInstance()
(No implicit dependencies #160)serializer
) (77490f6)StreamOffset
) (New directive@type
#157, No implicit dependencies #160, Type registration update (@type
) #161)spa
) (No implicit dependencies #160)The text was updated successfully, but these errors were encountered: