FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Deprecate cbook.warn_deprecated and move internal calls to _api.warn_deprecated by timhoffm · Pull Request #19162 · matplotlib/matplotlib · GitHub

Repository navigation

Deprecate cbook.warn_deprecated and move internal calls to _api.warn_deprecated - #19162

Merged
timhoffm merged 1 commit into
matplotlib:masterfrom
timhoffm:change-warn_deprecated-usage
Jan 5, 2021
Merged

timhoffm merged 1 commit into
matplotlib:masterfrom
timhoffm:change-warn_deprecated-usage

Conversation

timhoffm commented Dec 22, 2020 •
edited
Loading

Copy link
Copy Markdown
Member

PR Summary

Followup of moving all API helpers to matplotlib._api (#18657).

Builds on top of #19151.

  • still needs the actual deprecation.

timhoffm changed the title Change warn deprecated usage Deprecate cbook.warn_deprecated and move internal calls to _api.warn_deprecated Dec 22, 2020
timhoffm added this to the v3.4.0 milestone Dec 22, 2020
timhoffm force-pushed the change-warn_deprecated-usage branch 3 times, most recently from 2a1a943 to ecc851c Compare December 27, 2020 23:31
timhoffm marked this pull request as ready for review December 28, 2020 00:32
Comment thread lib/matplotlib/__init__.py Outdated

jklymak left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Anyone can merge, but I won't if @timhoffm wants to address @QuLogic's suggestion...

timhoffm force-pushed the change-warn_deprecated-usage branch from ecc851c to 227c171 Compare January 5, 2021 19:56
timhoffm merged commit b2921c8 into matplotlib:master Jan 5, 2021
timhoffm deleted the change-warn_deprecated-usage branch January 5, 2021 21:00
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL