You need to install the grunt-cli package #941

Closed
wants to merge 3 commits into
from

3 participants

@daanporon

If you don't install the grunt-cli package you can't use grunt install on the command line

@daanporon daanporon You need to install the grunt-cli package
If you don't install the grunt-cli package you can't use grunt install
28733eb
@sindresorhus sindresorhus and 1 other commented on an outdated diff Feb 21, 2013
@@ -30,6 +30,7 @@ Install Yeoman@0.9.6 and Grunt
```
npm install -g yeoman
npm install -g grunt
@sindresorhus
Yeoman member
sindresorhus added a line comment Feb 21, 2013

should be a dep in package.json and not here.

@daanporon
daanporon added a line comment Feb 22, 2013

this was a pr for the express-stack ... shouldn't grunt also be a dep in the package.json then?

@sindresorhus
Yeoman member
sindresorhus added a line comment Feb 22, 2013

wasn't that what I asked for? I'm confused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@sindresorhus
Yeoman member

ping

@addyosmani
Yeoman member

@daanporon just another ping :)

@daanporon

i was just wondering if grunt-cli should be a dep in the package file ... why not grunt itself? grunt and grunt-clli are two completely different npm packages now ... you need grunt-cli be able to do command line commands like the 'grunt install' command which is explained in the README.

@sindresorhus
Yeoman member

grunt-cli should be global and not in package.json since that might confuse users, grunt should be though. Nothing more to it than that.

@daanporon

like this then? i didn't know which version of grunt i should use ... but i'm working here with 0.4.0

@sindresorhus
Yeoman member

Yes

@addyosmani addyosmani added a commit that referenced this pull request Apr 24, 2015
@addyosmani addyosmani Merge branch 'express-stack' of https://github.com/yeoman/yeoman into…
… express-stack

* 'express-stack' of https://github.com/yeoman/yeoman: (792 commits)
  Close GH-941: You need to install the grunt-cli package.
  fixes grunt file path with spaces issue
  add more pre-install instructions
  Adding clone information.
  Fix attempt #2 for crud generators
  Adding bug reporting note.
  Adding further instructions.
  Adding links, tweaks.
  Reverting changes in Yeoman 0.9.6.x which made their way into this branch.
  fixed/enhanced grunt install script, updated readme.md
  Markdown list fix.
  Adding warranty notes.
  Adding express-stack current version to branch.
  Update logic to find local grunt
  Initial integration with the generator system
  Update dependencies
  Convert docs to use fenced code blocks again
  fixed typo (missing space)
  Typo fix (than vs. then)
  Remove quickstart generator from readme
  ...
7ad677a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment