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

As CLI tool? #30

Open
mikecann opened this issue Dec 21, 2021 · 3 comments
Open

As CLI tool? #30

mikecann opened this issue Dec 21, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@mikecann
Copy link

mikecann commented Dec 21, 2021

Any reason why this isnt simply a CLI tool on npm? Also why the explicit calling of node? This is kind of counter-intuitive to people coming from the web ecosystem.

I would also suggest having the user optionally explicity provide the config as it confused me and it seems to confuse others too (judging by the forum and issues).

A more usual / preferred developer experience would be:

  1. npm install playcanvas-sync -g
  2. playcanvas-sync -c my-config.json watchAll
@willeastcott
Copy link
Contributor

I agree with this 100%. playcanvas-sync should definitely be distributed and work as you describe IMHO.

@yaustar yaustar added the enhancement New feature or request label Dec 23, 2021
@zachpeterpaul
Copy link
Contributor

After 'npm install' from the project folder you can run it without 'node' or '.js'. The example command in the readme is for less experienced users who may not want to install it. './pcwatch.js' would also work.

@willeastcott
Copy link
Contributor

@zachpeterpaul Not on Windows, it doesn't.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants