-
Notifications
You must be signed in to change notification settings - Fork 6
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
Purpose of this bundle? #11
Comments
Sorry for late reply, missed the notification. Unfortunatelly we didn't have time to play with Syfmony 4 yet :/
I think more info should be able to provide @jakubkulhan who already used it with symfony 4 and is original author of this bundle :) |
Yep, in Symfony 4 this bundle only adds annotation-based property/method autowiring & scalar container parameter injection. |
Both of these features are a bad practice in my opinion. By bad practice I mean that it should not be specified in the classes by annotations but in config instead. The class is not supposed to know where the values are from or if they are autowired. Therefore I don't want this package in my application. It might be a good idea to release a new version of BunnyBundle without dependency on this one. I'll probably use EnqueueBundle anyway so it doesn't matter to me but you still might want to consider it. Thanks for your answers. |
I understand that this bundle is from the time when Symfony didn't have autowiring, autoregistration and other advanced featured. But symfony does have these features now. From the readme I'm not sure if this bundle still has some value or not in Symfony 4 environment. Can you give me some info about this?
I was looking for an integration of bunny to symfony and your bunny bundle depends on this one so I'd like to know what is it about before I install it.
The text was updated successfully, but these errors were encountered: