The Wayback Machine - https://web.archive.org/web/20201012071710/https://github.com/github/docs/pull/365
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add tip about nodenv permissions #365

Open
wants to merge 1 commit into
base: main
from
Open

add tip about nodenv permissions #365

wants to merge 1 commit into from

Conversation

@zeke
Copy link
Member

@zeke zeke commented Oct 9, 2020

This tiny PR adds a tip about overcoming nodenv permissions warnings.

Copy link
Contributor

@runleonarun runleonarun left a comment

Thanks Zeke!


```sh
# You should change the ownership of these directories to your user.
sudo chown -R $(whoami) /usr/local/sbin

This comment has been minimized.

@runleonarun

runleonarun Oct 9, 2020
Contributor

One thing that gave me pause was whether to replace $(whoami) with my username or just the whoami part. It might be worth calling out for people less familiar with the CLI.

Suggested change
sudo chown -R $(whoami) /usr/local/sbin
sudo chown -R USERNAME /usr/local/sbin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.
Morty Proxy This is a proxified and sanitized view of the page, visit original site.