Remove support for old-style multipolygons #2610

Merged
merged 1 commit into from Apr 17, 2017

Conversation

Projects
None yet
2 participants
@pnorman
Collaborator

pnorman commented Apr 16, 2017

No description provided.

@pnorman pnorman added the lua label Apr 16, 2017

@pnorman pnorman requested a review from matthijsmelissen Apr 16, 2017

@imagico

This comment has been minimized.

Show comment
Hide comment
@imagico

imagico Apr 16, 2017

Collaborator

This seems a good idea to me. Removal of old style tagging is continuing to be very fast so it is unlikely that a significant number of such features will remain when this change goes live even in the most optimistic scenario. Of course the fairly mechanical re-tagging happening will produce a lot of factually incorrect data but this will not in any way influence the sensibility of this change.

Collaborator

imagico commented Apr 16, 2017

This seems a good idea to me. Removal of old style tagging is continuing to be very fast so it is unlikely that a significant number of such features will remain when this change goes live even in the most optimistic scenario. Of course the fairly mechanical re-tagging happening will produce a lot of factually incorrect data but this will not in any way influence the sensibility of this change.

@pnorman pnorman merged commit 799aebf into gravitystorm:lua Apr 17, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@pnorman

This comment has been minimized.

Show comment
Hide comment
@pnorman

pnorman Apr 20, 2017

Collaborator

I'm not sure why this showed up in the diff.

Collaborator

pnorman commented on scripts/lua/test.lua in cb23299 Apr 20, 2017

I'm not sure why this showed up in the diff.

@pnorman pnorman deleted the pnorman:lua/no_old_style_mps branch May 10, 2017

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