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

Make cbook safe to import while removing duplicate is_string_like; #1762

Merged
merged 2 commits into from Feb 19, 2013

Conversation

efiring
Copy link
Member

@efiring efiring commented Feb 18, 2013

closes #1019, replaces #1662.

from the Python Cookbook -- hence the name cbook
from the Python Cookbook -- hence the name cbook.

This module is in transition, with the intention of migrating
Copy link
Member

Choose a reason for hiding this comment

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

This paragraph is the only potential bone of contention for me. Has there been general agreement on transitioning cbook? FWIW I've read #1647 and the relevant mailing list discussion and still haven't found a definitive strategy - personally, I think anything of that scale should be covered by a MEP with a full transition plan.

On the basis of my comments, I wonder if we are better off not pre-empting that work here, and simply removing this paragraph?

Copy link
Member

Choose a reason for hiding this comment

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

Moving the methods to a newly named module is debattable, but I think deprecating unused methods should still be done. I'm a a bit swamped with work right now, but I'll submit a new PR soon.

@pelson
Copy link
Member

pelson commented Feb 19, 2013

Other than my comment, 👍

Simplify module docstring, removing mention of possible future changes.
efiring added a commit that referenced this pull request Feb 19, 2013
Make cbook safe to import while removing duplicate is_string_like;
@efiring efiring merged commit 433da1a into matplotlib:master Feb 19, 2013
@efiring efiring deleted the cbook_safe_import branch February 19, 2013 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Repeated is_string_like function
3 participants