search

divec / ll

Two-way parallel translation
GNU Affero General Public License v3.0
3 stars 5 forks source link

Remove grunt commands from README.md #23

Closed catrope closed 5 years ago

catrope commented 5 years ago

In both repos, running npm install also runs grunt, so it doesn't need to be run separately.

divec commented 5 years ago

Maybe I'm being stupid, but "npm install" doesn't run grunt for me – is it only the first time you do "npm install"?

edg2s commented 5 years ago

It does for me and should according to https://docs.npmjs.com/misc/scripts

prepare: Run both BEFORE the package is packed and published, on local npm install without any arguments, and when installing git dependencies (See below). This is run AFTER prepublish, but BEFORE prepublishOnly.

Maybe you have an old version of npm?

divec commented 5 years ago

We discussed, and decided to decline this because with the old npm in ubuntu 18.04 (npm version 3.5.2), npm install doesn't run grunt for you.