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

Merged
merged 2 commits into from
Oct 14, 2020
Merged

add tip about nodenv permissions #365

merged 2 commits into from
Oct 14, 2020

Conversation

zeke
Copy link
Contributor

@zeke zeke commented Oct 9, 2020

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

@zeke zeke requested a review from runleonarun October 9, 2020 20:28
@zeke zeke added the engineering Will involve Docs Engineering label Oct 9, 2020
Copy link
Contributor

@runleonarun runleonarun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Zeke!


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

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whoami is actually a command on Unix systems. When you run it, it returns your username. The command suggested by the nodenv output will evaluate that on the fly, so sudo chown -R $(whoami) /usr/local/sbin becomes sudo chown -R leona /usr/local/sbin or similar.

@chiedo chiedo added engineering Will involve Docs Engineering and removed engineering Will involve Docs Engineering labels Oct 14, 2020
@zeke zeke merged commit 923c7e4 into main Oct 14, 2020
@zeke zeke deleted the add-nodenv-tip branch October 14, 2020 17:53
@github-actions
Copy link
Contributor

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours.

If you haven't already, you can add yourself to the list of contributors by creating a new comment in this PR using these instructions. Thanks again! ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engineering Will involve Docs Engineering
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants