-
Notifications
You must be signed in to change notification settings - Fork 2.3k
chore(scripts): clean up package scripts #3794
Conversation
cnishina
commented
Dec 5, 2016
- npm start - installs dependencies and starts the testapp
- npm run website - installs dependencies and starts the website
- cleanup testapp package scripts that are not used
- npm run format - shortcut for 'gulp format' for contributors that failed format check test
"tsc": "tsc", | ||
"tsc:w": "tsc -w", | ||
"lite": "lite-server" | ||
"install_testapp": "npm install && npm run tsc", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be postinstall
. Obviously we're installing the testapp, since we're in that folder. And postinstall
isn't a problem since we never publish the testapp.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, postinstall
would be better. Changed.
"lite": "lite-server" | ||
"install_testapp": "npm install && npm run tsc", | ||
"start": "npm run install_testapp && node scripts/web-server.js", | ||
"start_detach": "node scripts/web-server.js &", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's weird that start
does an install but start_detach
doesn't.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, what is the point of this command? Presumably people could do npm start &
on their own?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing install_testapp
from start
in favor of npm install
. Removing start_detach
since its only used in Travis.
47bd071
to
cec1541
Compare
- npm start - installs dependencies and starts the testapp - npm run website - installs dependencies and starts the website - cleanup testapp package scripts that are not used - npm run format - shortcut for 'gulp format' for contributors that failed format check test
cec1541
to
6fd92b3
Compare