Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

chore(yarn): rely on Travis built-in Yarn support, update Yarn in Jenkins #15851

Merged
merged 1 commit into from
Mar 22, 2017

Conversation

mgol
Copy link
Member

@mgol mgol commented Mar 22, 2017

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Chore

What is the current behavior? (You can also link to an open issue here)
We install an old Yarn verison manually.

What is the new behavior (if this is a feature change)?
On Travis we rely on built-in Yarn support and we only cache the Yarn cache, not node_modules. We install a new Yarn verison manually on Jenkins; the location of the install script changed.

Does this PR introduce a breaking change?
No.

Please check if the PR fulfills these requirements

Other information:

@mgol mgol self-assigned this Mar 22, 2017
@mgol
Copy link
Member Author

mgol commented Mar 22, 2017

This PR blocks #15843.

…kins

On Travis we now rely on built-in Yarn support and we only cache the Yarn cache,
not node_modules. This creates a more stable environment as we don't install
over previous node_modules state but we still won't download packages from the
internet in the second run for the same yarn.lock as Yarn takes packages from
its local cache if they exist there.

We install a new Yarn verison manually on Jenkins; the location of the install
script changed.
@mgol mgol merged commit 3d0abff into angular:master Mar 22, 2017
@mgol mgol deleted the yarn branch March 22, 2017 21:49
mgol added a commit that referenced this pull request Mar 22, 2017
…kins

On Travis we now rely on built-in Yarn support and we only cache the Yarn cache,
not node_modules. This creates a more stable environment as we don't install
over previous node_modules state but we still won't download packages from the
internet in the second run for the same yarn.lock as Yarn takes packages from
its local cache if they exist there.

We install a new Yarn verison manually on Jenkins; the location of the install
script changed.

Closes #15851
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.

3 participants