Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

Add sudo to setup instructions in README.md #29

Closed
wants to merge 1 commit into from

Conversation

tlshaheen
Copy link

Prevents future installers from running into #23

Prevents future installers from running into #23
@mikeosunajr
Copy link

I think it is generally considered bad practice to use sudo to install npm packages. Our setup script should not require sudo.

@bolinfest
Copy link
Contributor

I think that something else in your toolchain was, arguably erroneously, installed with sudo if you need sudo to run Nuclide's setup script. As such, we will not take this change.

@bolinfest bolinfest closed this Jun 4, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants