-
Notifications
You must be signed in to change notification settings - Fork 309
feat: merge Broker(context=...) and FastStream(context=...)
#2705
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
base: main
Are you sure you want to change the base?
Conversation
|
The current implementation merges the broker context into the application context, so a single context will be shared between multiple brokers. If the current version is accepted, I would further supplement the documentation. |
477550f to
7350ef9
Compare
Description
Fixes #2693
Changes
Checklist
just lintshows no errors)just test-coveragejust static-analysis