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

Update README.md #1774

Closed
wants to merge 1 commit into from
Closed

Update README.md #1774

wants to merge 1 commit into from

Conversation

Projjol-zz
Copy link

Add yarn as a dependency in the README file

Add yarn as a dependency in the README file
@luclu
Copy link
Contributor

luclu commented Mar 20, 2017

Thank you for the PR, @Projjol!

The yarn community does advice to refrain from installing yarn via npm (see here):

Note: Installation via npm is generally not recommended. npm is non-deterministic, packages are not signed, and npm does not perform any integrity checks other than a basic SHA1 hash, which is a security risk when installing system-wide apps.

For these reasons, it is highly recommended that you install Yarn through the installation method best suited to your operating system.

Although yarn is listed as a dependency in the Readme, it is not very clear.

I will take a look at it today and rephrase the paragraph. Thanks again for pointing this out!

@luclu luclu closed this Mar 20, 2017
@lock
Copy link

lock bot commented Mar 31, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked and limited conversation to collaborators Mar 31, 2018
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