Fix namespace(@mapbox) issue in tilelive.auto

mapbox / tilelive

fast interface to tiles with pluggable backends - NOT ACTIVELY MAINTAINED

BSD 3-Clause "New" or "Revised" License

531 stars 108 forks source link

Fix namespace(@mapbox) issue in tilelive.auto #191

Closed jingsam closed 7 years ago

coveralls commented 7 years ago

Coverage remained the same at 97.685% when pulling ae2db4fc4abfbd0197532774136c15c87a415d00 on jingsam:patch-3 into 89771fa32b27d63031cfb8fe103f90333af64d12 on mapbox:master.

jingsam commented 7 years ago

The tests seem unstable, success on node 0.10 and node 4, failed on node 0.12 and node 5

tcql commented 7 years ago

lets figure out what's up the with the tests and I'll get this merged

mapsam commented 7 years ago

The tests appear to be giving different results when run multiple times. Not great, but shouldn't block this fix. Retrying them a couple times gives us 🍏 Feel free to pull the trigger @tcql!

tcql commented 7 years ago

yea seems like the test indeterminism isn't related to this change 👍 thanks for the fix @jingsam