search

juliangruber / browser-run

Run code inside a browser from the command line
447 stars 62 forks source link

Moved browserify to dev-dependencies and added .npmignore #13

Closed rubennorte closed 9 years ago

rubennorte commented 9 years ago

Fixes #12

juliangruber commented 9 years ago

oherwise lgtm!

juliangruber commented 9 years ago

thanks! published as 2.1.0

rubennorte commented 9 years ago

Great! Thank you.

We've reduced that size of the dependency from 42 to 16MB. We just need to wait until browser-launcher updates plist so the whole package can be reduced to 2MB.

ainthek commented 9 years ago

CLI help not working withou devDependencies now:

browser-run --help module.js:338 throw err; ^ Error: Cannot find module 'browserify' at Function.Module._resolveFilename (module.js:336:15) at Function.Module._load (module.js:278:25) at Module.require (module.js:365:17) at require (module.js:384:17) at Object. (c:\Users...\AppData\Roaming\npm\node_modules\browser-run\index.js:5:18) at Module._compile (module.js:460:26) at Object.Module._extensions..js (module.js:478:10) at Module.load (module.js:355:32) at Function.Module._load (module.js:310:12) at Module.require (module.js:365:17)

juliangruber commented 9 years ago

moved back to deps and published as 2.1.1