search

mapbox / mts-data-sync

Simple publishing pipeline for the Mapbox Tiling Service
BSD 2-Clause "Simplified" License
71 stars 16 forks source link

Publish to NPM #12

Open tmcw opened 3 years ago

tmcw commented 3 years ago

The installation instructions for this utility seem odd relative to other Node-based utilities. Typically I'd expect to see something like

npm install -g mts-data-sync

To install a global utility, rather than cloning and linking. That would give the utility the benefit of only installing on tagged releases rather than whatever is in the main branch of the repository. Is there some reason why this repository uses a different technique?

jbranigan commented 3 years ago

I agree that this should be an NPM package. Modified the title of this issue to reflect that intent.

james-innes commented 1 year ago

For the other API endpoints there is a section in the docs telling you which SDK's have been made for it. For example geocoding. However for the tilesets API docs instead of referencing the generic JavaScript driver it points here to a NodeJs only (not browser) driver that seems a little clunky given you can't just npm install it. However the tilesets service does seem to be exposed in the generic Javascript SDK and documented here. So unless I'm missing something then most people would probably prefer to use that than this repo.