-
Notifications
You must be signed in to change notification settings - Fork 469
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
Change base namespace to PhpAmqpLib #332
Comments
This would be really big BC break, TBH. I'm not sure it's worth it... at least for now. IMO would be better to get "everything" fixed/merged that is waiting etc. and then think after that about such moves. Just my 2 cents ;) |
@stloyd Yeah, that is true. It's fine if it's not something to do now, just something to throw in the backlog. |
👍 for doing this in next major release (2.0) |
Also we can manage some migration script as well. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
It would be very big and breaking change. Not planned any time soon AFAIK. |
Now that the
RabbitMqBundle
is a part of thephp-amqplib
organization, it might be good to change the base namespace fromOldSound
toPhpAmqpLib
. The problem is that this would be a BC break, so you would have to make that happen during the next major release. But just wanted to start the discussion here...The text was updated successfully, but these errors were encountered: