Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.6k
[DI] add tagscontainer.preload
/.no_preload
to declare extra classes to preload/services to not preload#36195
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
82543d0
tof15656e
CompareSo |
@michaljusiega AFAIU, the class_exists were solving 2 issues:
This PR provides an alternative solution to the first issue (though |
container.preload
/.no_preload
to declare extra classes to preload/services to not preloadf15656e
to45dd84f
Comparenicolas-grekas commentedMar 25, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I updated the PR to now also provide a @michaljusiega: what@stof said: |
b62ee86
tob075b40
Compare@nicolas-grekas,@stof I understand. Thanks! |
Uh oh!
There was an error while loading.Please reload this page.
c0121bf
to966ad2d
Compare…ses to preload/services to not preload
Thank you@nicolas-grekas. |
To allow fine-grained declaration of sidekick classes in DI extensions.
Follows#36103