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

MEP12: Example clean-up for reference #2474

Merged
merged 21 commits into from Jan 14, 2014

Conversation

tonysyu
Copy link
Contributor

@tonysyu tonysyu commented Sep 29, 2013

This PR is meant as a reference for clean-ups that should take place as part of the Gallery clean ups outlined in MEP12.

@NelleV
Copy link
Member

NelleV commented Oct 3, 2013

That looks good to me. Are we sure that there are no explicit links to this example in the documentation ?

Also, can you squash some of those commits. That's a long list for the size of the patch :)

Else, I'm 👍

@WeatherGod
Copy link
Member

I am against squashing this. That list is an excellent bulletted list of
the kinds of changes that are needed. And it makes it very easy to
reference an example of each.

@NelleV
Copy link
Member

NelleV commented Oct 3, 2013

Squashing commits doesn't mean removing the content of the commit messages.

@tonysyu
Copy link
Contributor Author

tonysyu commented Oct 4, 2013

Actually, I wanted to be as specific as possible with each commit. There were a few I might have wanted to group together, but for the most part each one gives a good, isolated description of different types of changes. This is important because I use these commits as concrete examples in the MEP (which links directly to commits).

And yes, as far as I can tell, none of the docs link to this example.

@tacaswell
Copy link
Member

Is there any reason this has not been merged?

pelson added a commit that referenced this pull request Jan 14, 2014
@pelson pelson merged commit d81498f into matplotlib:master Jan 14, 2014
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.

None yet

5 participants