search

esbullington / react-d3

Modular React charts made with d3.js
https://reactiva.github.io/react-d3-website/
MIT License
1.76k stars 179 forks source link

missing script: build #314

Closed ernest-rhinozeros closed 8 years ago

ernest-rhinozeros commented 8 years ago

If I try to run the minified script I get the error message:

npm ERR! missing script: build

The scripts section of the package.json looks like this:

"scripts": {
    "env": "env",
    "test": "gulp test",
    "watch": "gulp watch",
    "minified": "gulp release && npm run build && NODE_ENV=production browserify -t [ reactify --es6 ] --standalone rd3 ./build/cjs/index.js | uglifyjs -c > dist/public/js/react-d3.min.js"
  },

So there is no build script and there never was AFAICS (I was looking at the history of package.json and apparently https://github.com/esbullington/react-d3/commit/f9f7244458aa0452f0bf0b1dab3ba8ad810c75bf#diff-b9cfc7f2cdf78a7f4b91a753d10865a2 introduced the minified script but there was no build script).

Removing the npm run build && and running npm run minified will work and it seems that everything necessary is created but I don't know if I'm missing something.

yang-wei commented 8 years ago

Removing the npm run build && and running npm run minified will work and it seems that everything necessary is created but I don't know if I'm missing something.

You are correct. Mind to send us a pr ?

ernest-rhinozeros commented 8 years ago

I don't know if just removing the npm run build is alright, but I'll commit the version without it. @esbullington should know if there's missing something from the npm run build script.