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

upgrade to delaunator@4 #63

Closed
wants to merge 1 commit into from
Closed

upgrade to delaunator@4 #63

wants to merge 1 commit into from

Conversation

Fil
Copy link
Member

@Fil Fil commented May 24, 2019

fixes #50

removes delaunay.outedges from the API, replaced with the private _hullIndex

fixes #50

removes delaunay.outedges from the API, replaced with the private _hullIndex
@mourner
Copy link
Collaborator

mourner commented Jun 4, 2019

@Fil let's merge then? Or did you plan to get the degenerate cases handling into the same major version?

@Fil
Copy link
Member Author

Fil commented Jun 4, 2019

Depends on your feedback on mapbox/delaunator#47 ; if negative, then we can merge this and find a better way to solve #64. If positive, then we can make a more interesting major version.

@mourner
Copy link
Collaborator

mourner commented Jun 18, 2019

@Fil that one is merged, but I'm thinking of also merging in mapbox/delaunator#48 to cram this into a semver major too, and then we can release new major versions of both Delaunator and d3-delaunay.

@Fil
Copy link
Member Author

Fil commented Jun 18, 2019

Yes let's move forward with everything

@Fil Fil changed the title upgrade to delaunator@3 Jun 19, 2019
@Fil Fil changed the title upgrade to delaunator@34 Jun 19, 2019
@Fil
Copy link
Member Author

Fil commented Jun 19, 2019

Superseded by #64

@Fil Fil closed this Jun 19, 2019
@Fil Fil deleted the delaunator3 branch July 9, 2019 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants