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

Handle Mapnik errors when creating VT #82

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

GretaCB
Copy link
Contributor

@GretaCB GretaCB commented Apr 21, 2016

With recent Vector Tile v2 spec updates, Mapnik is now throwing on errors that previously did not throw.

This PR helps handle these errors gracefully with a more explicit error code EINVALID for expected errors (errors that do not need to throw).

cc @springmeyer @BergWerkGIS @rclark

@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 94.163% when pulling 6ccb6e4 on handle-error into 15e4501 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants