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

Update README.md#1774

Closed
Projjol-zz wants to merge 1 commit intoethereum:developfrom
Projjol-zz:patch-1
Closed

Update README.md#1774
Projjol-zz wants to merge 1 commit intoethereum:developfrom
Projjol-zz:patch-1

Conversation

@Projjol-zz
Copy link
Copy Markdown

Add yarn as a dependency in the README file

Add yarn as a dependency in the README file
@luclu
Copy link
Copy Markdown
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
Copy Markdown

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