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

build: fix master build by updating the yarn.lock after race merge #18074

Merged
merged 1 commit into from Apr 30, 2019

Conversation

Projects
None yet
2 participants
@MarshallOfSound
Copy link
Member

commented Apr 30, 2019

There was a small race here between my yarn PR landing and another dependency being bumped so when they both got merged the lock file was out of date. This is my bad, should have rebased before merging. This will fix it up for good now though 👍

Notes: no-notes

@MarshallOfSound

This comment has been minimized.

Copy link
Member Author

commented Apr 30, 2019

As soon as a checkout passes I'm gonna merge this is green-ify master 👍

@MarshallOfSound MarshallOfSound merged commit 4e5a094 into master Apr 30, 2019

4 of 6 checks passed

appveyor: win-ia32-testing Waiting for AppVeyor build to complete
Details
appveyor: win-x64-testing Waiting for AppVeyor build to complete
Details
Semantic Pull Request ready to be squashed
Details
appveyor: win-ia32-testing-pr AppVeyor build succeeded
Details
appveyor: win-x64-testing-pr AppVeyor build succeeded
Details
release-notes Release notes found
@release-clerk

This comment has been minimized.

Copy link

commented Apr 30, 2019

No Release Notes

@MarshallOfSound MarshallOfSound deleted the fix-master branch Apr 30, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.