Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Coding guidelines concerning the API#14555
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
doc/devel/contributing.rst Outdated
| `MatplotlibDeprecationWarning`. There are a number of helper tools for this: | ||
| - Use `.cbook.warn_deprecated()` for general deprecation warnings. | ||
| - Use the decorator ``@cbook.warn_deprecated`` to deprecate whole functions. |
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.
cbook.deprecated
to deprecate classes, functions, methods, or properties.
doc/devel/contributing.rst Outdated
| `MatplotlibDeprecationWarning`. There are a number of helper tools for this: | ||
| - Use `.cbook.warn_deprecated()` for general deprecation warnings. | ||
| - Use the decorator ``@cbook.warn_deprecated`` to deprecate classes, |
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.
@cbook.deprecated :)
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.
Just one typo, but anyone can merged after it is fixed and CI passes.
PR Summary
Alternative proposal to#14545, as discussed therein.