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

chore: add yarn engines entry #437

Merged
merged 1 commit into from
Nov 7, 2018
Merged

Conversation

bmdalex
Copy link
Collaborator

@bmdalex bmdalex commented Nov 6, 2018

package.json engines entry for yarn (chore)

Motivation

yarn 1.10.0 introduces an integrity field with sha512 authentication to yarn.lock to preserve the integrity of every installed package before its code is executed.

A machine with a version of yarn lower than 1.10.0 would automatically remove the

integrity sha512-XXX==

entries from yarn.lock causing unwanted changes.

Resolution

This PR that forces devs to upgrade to yarn 1.10.0 or above by adding the engines entry to package.json.

Note: On Mac, the most reliable way to upgrade yarn to latest stable version is:

curl -o- -L https://yarnpkg.com/install.sh | bash

@bmdalex
Copy link
Collaborator Author

bmdalex commented Nov 6, 2018

fyi @levithomason @layershifter

@bmdalex bmdalex force-pushed the chore/add-yarn-engines-entry branch from af9726a to 9c259cb Compare November 6, 2018 14:31
@codecov
Copy link

codecov bot commented Nov 6, 2018

Codecov Report

Merging #437 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #437   +/-   ##
=======================================
  Coverage   91.84%   91.84%           
=======================================
  Files          41       41           
  Lines        1337     1337           
  Branches      168      168           
=======================================
  Hits         1228     1228           
  Misses        105      105           
  Partials        4        4

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2dd320...a950fcf. Read the comment docs.

Copy link
Member

@layershifter layershifter left a comment

Choose a reason for hiding this comment

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

Looks correct to me:
https://yarnpkg.com/lang/en/docs/package-json/#toc-engines

You can see an example issue in #411:
image

@bmdalex bmdalex force-pushed the chore/add-yarn-engines-entry branch from 9c259cb to a950fcf Compare November 7, 2018 18:05
@bmdalex bmdalex merged commit fdbcde7 into master Nov 7, 2018
@bmdalex bmdalex deleted the chore/add-yarn-engines-entry branch November 7, 2018 18:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants