Skip to content
This repository has been archived by the owner on Oct 10, 2024. It is now read-only.

adds --force to npm install yarn to fix circle ci build. #300

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

case7026
Copy link

@case7026 case7026 commented Jun 27, 2020

Objective:

  • Fix circle ci builds failing

Changes

  • Add --force to ensure that if yarn is already installed in the build container's base image that we ensure that we reinstall with an appropriate version

Suggestions / Question for maintainers

  • Is there a specific reason to pin yarn to 1.9.4? If not, we should consider removing the explicit yarn version declaration and entirely remove this step from the CI pipeline to simplify & speed up the build process. I will be opening another pull request with this change and linking it here in an edit shortly. That PR is located here.

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.

1 participant