Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade Yarn to v1.6 #21954

Merged
merged 3 commits into from Apr 26, 2018
Merged

Upgrade Yarn to v1.6 #21954

merged 3 commits into from Apr 26, 2018

Conversation

islemaster
Copy link
Contributor

@islemaster islemaster commented Apr 19, 2018

Upgrades our Yarn dependency from v0.23.2 to v1.6.0.

We last did this about a year ago. This change will require developers to update their local yarn version, probably by following the appropriate steps in SETUP.md again. I just ran the following to update:

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

Copy link
Contributor

@caleybrock caleybrock left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@@ -1,23 +0,0 @@
#!/usr/bin/env bash
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Surprise! We were no longer calling this file anywhere, so we've been using yarn 0.27.5 in our Circle builds, probably since the Circle 2.0 upgrade. This probably accounts for some of the weird yarn.lock issues we've run into.

image

@islemaster
Copy link
Contributor Author

PTAL

@@ -76,6 +79,10 @@ jobs:
command: |
bundle check --path=/home/circleci/project/vendor/bundle || bundle install --deployment --path=/home/circleci/project/vendor/bundle --jobs=4 --retry=3 --without ''

- run:
name: set yarn version
command: sudo apt-get install yarn=1.6.0-1
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we set this up in our docker image instead of installing it every time?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point. Looks like we'd change that here:

# add yarn
RUN curl -sS https://dl.yarnpkg.com/debian/pubkey.gpg | apt-key add - && \
echo "deb https://dl.yarnpkg.com/debian/ stable main" | tee /etc/apt/sources.list.d/yarn.list

But I potentially need @wjordan to push the updated image to Docker hub?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Discussed in person. I'm going to go ahead and merge this with the install in the Circle config (takes about 4s) and expect follow-up work to move our docker image someplace with shared ownership, at which time we can move this up to the image.

Copy link
Contributor

Choose a reason for hiding this comment

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

Did this ever happen?

follow-up work to move our docker image someplace with shared ownership

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I suspect it did - the yarn version is set in the dockerfile here:

@uponthesun may know more about where our image lives / how to update it.

Copy link
Contributor

@aoby aoby left a comment

Choose a reason for hiding this comment

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

LGTM, but I'm not very familiar with the circle config

@islemaster islemaster merged commit 48e8e40 into staging Apr 26, 2018
@islemaster islemaster deleted the yarn-1.6 branch April 26, 2018 23:24
This was referenced Apr 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants