-
Notifications
You must be signed in to change notification settings - Fork 188
Fix autowiring #913
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix autowiring #913
Conversation
93ed770
to
4122bbe
Compare
Travis-ci was failing because of an unrelated while trying to download something, so I re-pushed the branch. |
Codecov Report
@@ Coverage Diff @@
## 7.0 #913 +/- ##
=========================================
Coverage 83.16% 83.16%
Complexity 413 413
=========================================
Files 39 39
Lines 1360 1360
=========================================
Hits 1131 1131
Misses 229 229
Continue to review full report at Codecov.
|
can you please fix also for v6 bundle and symfony 4.4? thanks!
|
Didn't notice it is also a problem for v6, will cherry-pick it with the next release of v6 |
@saimaz I already made the PR 👍 |
It's not released yet. I will release it in the evening. |
dont work even with EDIT: and the same problem is with version 7 with Symfony 4.4 |
@patie clearing the cache might not be enough. Delete the symfony |
deleted but dont work anyway, now do refactor on our project for upgrade to sf5 |
Autowiring doesn't work for me without
autowire: true
inservices.yml
and using theContainer
as typehint doesn result in the following message: