-
-
Notifications
You must be signed in to change notification settings - Fork 245
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
Feat/659 unbound queues #662
Conversation
Should run green now! |
Just tried it out and spotted that I am missing the ability to bind it to an exchange like this. Gonna take another look on how to do that. |
@WonderPanda the queues are also bindable to an exchange with routing keys. I also already tested it locally! Would be glad about a review. |
@underfisk do you also have review & merge rights? |
Hi, i do have the review & merge rights. As soon as i have some free time i'll try to switch context and review the PR, meawhile i'll approve for CI to re-run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general LGTM, it looks like a nice feature to have.
The types could be improved tho, for example you could have the RabitMQQueConfig
being built on top of the underlying types instead of using any types but it could be a follow up improvement
@fermentfan This should be available in Please try it out and let us know if its working for you |
This introduces unbound queues to be added to the module definition like described in #659. I hope I found everything important :)