-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Add dispatcher argument to Updater #1484
Conversation
Can you fix the open problems marked by CI? |
Fixed Flake8 and Codacy (by adding a property to |
pre-commit still gives a flake8 error |
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.
One small change requested
I'm also confused to why codecov isn;t running on this. |
We can ignore the codecov problem here. Ready to merge |
This allows to pass a pre-initialized
Dispatcher
instance toUpdater
, closing #1407.If the new
dispatcher
argument is used, attributes likebot
andjob_queue
are fetched from thedispatcher
rather than initializing them.In the docstring tried to make clear that proper initialization of the
dispatcher
is up to the user, i.e. it is not checked ifdispatcher.job_queue is not None
.Also added tests for creation with
dispatcher
. I hope those are enough …