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

V3.6.2 carto mvt variables #10

Merged
merged 5 commits into from
Jan 5, 2018
Merged

Conversation

rafatower
Copy link

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #10 into v3.6.2-carto will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff              @@
##           v3.6.2-carto      #10   +/-   ##
=============================================
  Coverage         96.22%   96.22%           
=============================================
  Files                42       42           
  Lines              8814     8814           
=============================================
  Hits               8481     8481           
  Misses              333      333
Impacted Files Coverage Δ
src/mapnik_map.cpp 99.92% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4589225...f19c6c3. Read the comment docs.

@rafatower rafatower merged commit 3f22928 into v3.6.2-carto Jan 5, 2018
@rafatower rafatower deleted the v3.6.2-carto-mvt-variables branch January 5, 2018 11:38
rafatower pushed a commit to CartoDB/tilelive-bridge that referenced this pull request Jan 9, 2018
Add some variables to be used later on by mapnik postgis input for token
substitution.

Note that the actual substitution depends on some changes in node-mapnik
and mapnik-vector-tile:

mapbox/mapnik-vector-tile#248
https://github.com/CartoDB/node-mapnik/blob/v3.6.2-carto/CHANGELOG.carto.md#362-carto1
CartoDB/node-mapnik#10
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