Skip to content

Commit

Permalink
Remove sudo from npm install bower and add a note
Browse files Browse the repository at this point in the history
Removed sudo from `npm install -g bower` and added a note for those struggling to use `npm install` without `sudo`. Also, added a warning listing possible threats.
  • Loading branch information
ryzokuken authored and jywarren committed Oct 9, 2016
1 parent 75b21ec commit f6255af
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,11 @@ You'll also need **bower** which is available through `npm`, part of `node.js`.

Once NPM is installed, you should be able to run:

`sudo npm install -g bower`
`npm install -g bower`

**NOTE:** If you're having permission issues, please see https://docs.npmjs.com/getting-started/fixing-npm-permissions

**WARNING:** Please refrain from using `sudo npm` as it's not only a bad practice, but may also put your security at a risk. For more on this, read https://pawelgrzybek.com/fix-priviliges-and-never-again-use-sudo-with-npm/

### phantomjs for javascript tests

Expand Down

0 comments on commit f6255af

Please sign in to comment.