Closed jchamberlain closed 7 months ago
ARolek
commented
9 months ago @jchamberlain could you please split this PR into 2 commits: 1 for the vendoring of geom (and go.mod updates) and then a second one for the code changes? Makes review a bit easier.
ARolek
commented
8 months ago @jchamberlain can you rebase against master? I just fixed the docker hub issue (and upgraded Go in the CI) so we can get the CI to pass. Then I can merge this in.
coveralls
commented
7 months ago | Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| cmd/tegola/cmd/cache/slippy.go | 18 | 20 | 90.0% | ||
| cmd/tegola/cmd/cache/seed_purge.go | 9 | 16 | 56.25% | ||
| server/handle_map_layer_zxy.go | 12 | 24 | 50.0% | ||
| provider/provider.go | 4 | 17 | 23.53% | ||
| <!-- | Total: | 51 | 85 | 60.0% | --> |
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| provider/provider.go | 1 | 54.29% | ||
| cmd/tegola/cmd/cache/cache.go | 2 | 23.56% | ||
| <!-- | Total: | 3 | --> |
| Totals | |
|---|---|
| Change from base Build 4d93140a4: | -0.05% |
| Covered Lines: | 6571 |
| Relevant Lines: | 14024 |
This addresses the second point in #951. The main changes impacting this project are in the
slippypackage, and include:slippy.Tiledirectly.Extent3857(),Extent4326()), thus requiring use a another tool for transformations.slippy.RangeFamilyAt(), thus requiring keeping around the old version of that function.