Skip to content
This repository has been archived by the owner on Jul 4, 2022. It is now read-only.

Added a solution for another installation case #36

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ameya-deshmukh
Copy link

Added the solution to use sudo along with yarn global installation in case adding yarn to the global PATH does not work.

I tried adding yarn to my system's path but to no avail. However, installing it globally using yarn with sudo worked perfectly.
Added another installation case and solution
README.md Outdated Show resolved Hide resolved
@ameya-deshmukh
Copy link
Author

@johnnymatthews done. Please merge now if possible, thanks!

Copy link
Contributor

@johnnymatthews johnnymatthews left a comment

Choose a reason for hiding this comment

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

I don't have merge rights, but this looks fine from my end :)

@Kayaba-Attribution
Copy link

I don't see the need to add this to the README, is my opinion tho

@ameya-deshmukh
Copy link
Author

I don't see the need to add this to the README, is my opinion tho

Fair enough I guess, although it'd be nice to include it as part of the README rather than browsing forums for solutions and wasting time, like I did. It's not even that long :) @JetJadeja please have a look, if it seems worth it, would appreciate you merging it.

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.

3 participants