Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Update README.md #327

Closed
wants to merge 1 commit into from

Conversation

akshay-waghmare
Copy link

No description provided.

@jkleinsc
Copy link
Member

jkleinsc commented Mar 3, 2016

@akiaksdon The comment you have added

(needs root access to install PhantomJs)

is not necessarily true, as it just depends how you have NPM setup. See here for more details:
https://docs.npmjs.com/getting-started/fixing-npm-permissions

@jkleinsc
Copy link
Member

@akiaksdon I've given this some more thought and I am going to merge with a slight refinement:
Depending on your npm permissions you might need root access to install PhantomJS2.

@jkleinsc jkleinsc closed this in 2a64b90 Mar 14, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants