/matplotlib

Fix #5670. No double endpoints in Path.to_polygon #5672

Merged
merged 3 commits into from Dec 15, 2015
+93 −19

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  v1.5.1
2 participants
@mdboom @tacaswell
@mdboom
Owner

mdboom commented Dec 14, 2015

No description provided.

@mdboom mdboom Fix #5670. No double endpoints in Path.to_polygon

Some checks were not successful

1 failing, 1 pending, and 1 successful checks
continuous-integration/travis-ci/pr — The Travis CI build failed
Details
coverage/coveralls — Coverage pending from Coveralls.io
Details
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
36ce4a7

mdboom added this to the Critical bugfix release (1.5.1) milestone Dec 14, 2015

mdboom added the needs_review label Dec 14, 2015

mdboom added some commits Dec 14, 2015

@mdboom mdboom Fix unit tests

Some checks were not successful

2 failing and 1 successful checks
continuous-integration/appveyor/pr — AppVeyor build failed
Details
continuous-integration/travis-ci/pr — The Travis CI build failed
Details
coverage/coveralls — Coverage remained the same at 68.253%
Details
66a4f88
@mdboom mdboom PEP8

Some checks were not successful

1 failing and 2 successful checks
coverage/coveralls — Coverage decreased (-0.008%) to 68.244%
Details
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
bf230fd
@mdboom
Owner

mdboom commented Dec 14, 2015

This is ready-to-merge.

@tacaswell tacaswell added a commit that referenced this pull request Dec 15, 2015

@tacaswell tacaswell Merge pull request #5672 from mdboom/double-endpoints 
Fix #5670. No double endpoints in Path.to_polygon
 a82ca89

@tacaswell tacaswell merged commit a82ca89 into matplotlib:master Dec 15, 2015
2 of 3 checks passed

2 of 3 checks passed

coverage/coveralls Coverage decreased (-0.008%) to 68.244%
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

tacaswell removed the needs_review label Dec 15, 2015

@tacaswell tacaswell added a commit that referenced this pull request Dec 15, 2015

@tacaswell tacaswell Merge pull request #5672 from mdboom/double-endpoints 
Fix #5670. No double endpoints in Path.to_polygon
 47dded2
@tacaswell
Owner

tacaswell commented Dec 15, 2015

backported to 1.5.x as 47dded2

This was referenced Jul 16, 2016

Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment