-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Deprecate plugins #2995
Deprecate plugins #2995
Conversation
dash/dash.py
Outdated
@@ -557,6 +557,12 @@ def __init__( # pylint: disable=too-many-statements | |||
if plugins is not None and isinstance( | |||
plugins, patch_collections_abc("Iterable") | |||
): | |||
warnings.warn( | |||
DeprecationWarning( | |||
"plugins will be removed from Dash init in dash 3.0 and will be replaced by" |
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.
- "Plugins" (capital 'P' at the start of the sentence)
- "Dash 3.0" (rather than 'dash 3.0')
Other than that, 👍 from me.
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.
👍 It's the plugins
keyword, let me change that.
dash/dash.py
Outdated
if long_callback_manager: | ||
warnings.warn( | ||
DeprecationWarning( | ||
"`long_callback_manager` is deprecated and will be remove in dash 3.0, " |
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.
"Dash 3.0" instead of "dash 3.0" for consistency please
Deprecate the
plugins
keyword from the Dash__init__
, will be removed in dash 3.0 in favor of a system that allows auto activation from pip installed package.I don't think the system was much used, the only documentation is from the
Dash.__init__
docstring.