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

Remove some unused imports; reword/remarkup some docstrings. #9861

Merged
merged 1 commit into from Nov 27, 2017

Conversation

Projects
None yet
5 participants
@anntzer
Copy link
Contributor

commented Nov 26, 2017

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is PEP 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

@dstansby dstansby added this to the v2.2 milestone Nov 26, 2017

Almost all vector drawing makes use of Paths somewhere in the drawing
pipeline.
The primary class for polyline handling in matplotlib is `Path`. Almost all
vector drawing makes use of `Path`\s somewhere in the drawing pipeline.

This comment has been minimized.

Copy link
@efiring

efiring Nov 26, 2017

Member

For my information, why is the backslash needed here? And should "matplotlib" be capitalized in the line above?

This comment has been minimized.

Copy link
@anntzer

anntzer Nov 26, 2017

Author Contributor

@anntzer anntzer force-pushed the anntzer:cleanups branch from 4817cc9 to 8dbaea6 Nov 26, 2017

@NelleV NelleV merged commit 3b55a8a into matplotlib:master Nov 27, 2017

8 checks passed

ci/circleci: docs-python27 Your tests passed on CircleCI!
Details
ci/circleci: docs-python35 Your tests passed on CircleCI!
Details
codecov/patch 100% of diff hit (target 50%)
Details
codecov/project/library 61.73% (target 50%)
Details
codecov/project/tests 98.86% remains the same compared to b3b91cc
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
lgtm analysis: Python No alert changes
Details

@anntzer anntzer deleted the anntzer:cleanups branch Nov 27, 2017

@QuLogic QuLogic modified the milestones: needs sorting, v2.2.0 Feb 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.