search

kubetail-org / loadjs

A tiny async loader / dependency manager for modern browsers (899 bytes)
MIT License
2.58k stars 150 forks source link

add npm scripts #32

Closed javoski closed 7 years ago

javoski commented 7 years ago

Using npm scripts may be better

amorey commented 7 years ago

Thanks! This is helpful. Taking a look at the npm scripts documentation another option is to collapse the build scripts into one "gulp" script which can be run this way:

$ npm run gulp -- <task-name>

What do you think about this approach?

javoski commented 7 years ago

Yeah. It's an another viable way, but typing less when running a task might be preferred.

amorey commented 7 years ago

Ok, thanks for the feedback! I kept the behavior of the build scripts interface the same but cleaned up the naming scheme. I also removed the port from the "serve" script to let the user choose their own script and I added a "gulp" escape hatch for developers who want it: https://github.com/muicss/loadjs/blob/master/package.json#L21-L28