Skip to content
This repository has been archived by the owner on Aug 5, 2020. It is now read-only.

Remove recommendation for installing as root #303

Merged
merged 1 commit into from
Oct 3, 2016
Merged

Conversation

threehams
Copy link
Collaborator

Reasons:

  • npm, for a while, recommended never running as root, and instead suggested brew / brew + nvm / some other non-root method of installation.
  • Nearly all other globally-installed npm packages don't include sudo as a recommendation.
  • sudo doesn't exist on Windows.
  • Installing Gluestick as root, while having a non-root Node installation, ends up mixing up permissions within the folder. Installing not as root will give a permissions error, and the user can add sudo to make it work. Overall, less damaging.

@christinebrass
Copy link
Contributor

👍

@christinebrass christinebrass merged commit 80639a7 into develop Oct 3, 2016
@christinebrass christinebrass deleted the remove-sudo branch October 3, 2016 14:54
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.

2 participants