Skip to content
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

Notifier: The docstring for notifier.add_observer() is incomplete. #7416

Closed
drew2a opened this issue May 10, 2023 · 1 comment
Closed

Notifier: The docstring for notifier.add_observer() is incomplete. #7416

drew2a opened this issue May 10, 2023 · 1 comment
Assignees

Comments

@drew2a
Copy link
Contributor

drew2a commented May 10, 2023

The "synchronous" parameter is not documented:

def add_observer(self, topic: FuncT, observer: FuncT, synchronous: Optional[bool] = None):
""" Add the observer for the topic.
Each callback will be added no more than once. Callbacks are called in the same order as they were added.
topic:
A callable which represents a "topic" to subscribe
observer:
A callable which will be actually called when notification is sent to the topic
synchronous:
A strategy of how to call the observer. If True,
"""

@qstokkink
Copy link
Contributor

The code references in this issue no longer exists and, therefore, this issue is no longer relevant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants