Skip to content

planar: Simplify shouldn't return polygons with < 3 points #77

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 30, 2019
Merged

planar: Simplify shouldn't return polygons with < 3 points #77

merged 1 commit into from
Sep 30, 2019

Conversation

ear7h
Copy link
Contributor

@ear7h ear7h commented Sep 30, 2019

Todo: add tests

@ear7h ear7h mentioned this pull request Sep 30, 2019
Merged
ARolek
ARolek approved these changes Sep 30, 2019
View reviewed changes
Copy link
Member

@ARolek ARolek left a comment

Choose a reason for hiding this comment

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

LGTM. @gdey would know if anything else needs to be considered.

gdey
gdey approved these changes Sep 30, 2019
View reviewed changes
Copy link
Member

@gdey gdey left a comment

Choose a reason for hiding this comment

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

LGTM

@gdey gdey merged commit 0cd65d8 into master Sep 30, 2019
@gdey gdey deleted the simplify-bug-fix branch September 30, 2019 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gdey gdey gdey approved these changes

@ARolek ARolek ARolek approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ear7h @gdey @ARolek