Skip to content

Begin deprecation of "hold": remove from examples#7515

Merged
NelleV merged 1 commit intomatplotlib:v2.xfrom
efiring:kill_hold
Nov 26, 2016
Merged

Begin deprecation of "hold": remove from examples#7515
NelleV merged 1 commit intomatplotlib:v2.xfrom
efiring:kill_hold

Conversation

@efiring
Copy link
Copy Markdown
Member

@efiring efiring commented Nov 25, 2016

This is a start on addressing #3070. The first commit should be uncontroversial. I will look into adding the deprecation warning in subsequent commits.

@efiring efiring changed the title Begin deprecation of "hold": remove from examples [WIP] Begin deprecation of "hold": remove from examples Nov 25, 2016
@efiring efiring added this to the 2.0 (style change major release) milestone Nov 25, 2016
@NelleV
Copy link
Copy Markdown
Member

NelleV commented Nov 26, 2016

Do you want to create one whole big PR or should we review/merge as these come in.
(We could merge this for example).

@efiring
Copy link
Copy Markdown
Member Author

efiring commented Nov 26, 2016

If you want to merge this, go ahead. I am working on stage 2 right now, but if this is merged I can easily make that another PR. Stage 2 is not so trivial, so it might require some revisions.

@efiring efiring changed the title [WIP] Begin deprecation of "hold": remove from examples [MRG] Begin deprecation of "hold": remove from examples Nov 26, 2016
@efiring
Copy link
Copy Markdown
Member Author

efiring commented Nov 26, 2016

This is actually independent of stage 2, so please go ahead and merge it whenever you like. I will make stage 2 a separate PR right now.

@NelleV NelleV merged commit fc204fa into matplotlib:v2.x Nov 26, 2016
@NelleV
Copy link
Copy Markdown
Member

NelleV commented Nov 26, 2016

Thanks!

@QuLogic QuLogic changed the title [MRG] Begin deprecation of "hold": remove from examples Begin deprecation of "hold": remove from examples Nov 26, 2016
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