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

Use modern geometry types for gazetteer backend #697

Merged
merged 3 commits into from Mar 4, 2017

Conversation

pnorman
Copy link
Collaborator

@pnorman pnorman commented Mar 3, 2017

Fixes #309

The test coverage of the gazetteer backend is lacking, but these are the type of changes which tend to cause obvious failures, not subtle ones.

@lonvia
Copy link
Collaborator

lonvia commented Mar 3, 2017

The PR has conflicts now, can you rebase?

There are more extensive tests in Nominatim, I will check that they still pass.

@pnorman
Copy link
Collaborator Author

pnorman commented Mar 3, 2017

The PR has conflicts now, can you rebase?

Done

Copy link
Collaborator

@lonvia lonvia left a comment

Choose a reason for hiding this comment

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

Tests run ok on Nominatim.

@pnorman pnorman merged commit ed78f96 into osm2pgsql-dev:master Mar 4, 2017
@pnorman pnorman deleted the gazetteer_types branch March 4, 2017 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants