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

Publish to NPM #12

Open
tmcw opened this issue Jan 20, 2021 · 2 comments
Open

Publish to NPM #12

tmcw opened this issue Jan 20, 2021 · 2 comments

Comments

@tmcw
Copy link

tmcw commented Jan 20, 2021

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 jbranigan changed the title Unusual installation instructions Publish to NPM Jan 21, 2021
@jbranigan
Copy link
Contributor

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

@james-innes
Copy link

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.

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

No branches or pull requests

3 participants