Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

Switch to use greenkeeper-lockfile-update for yarn.lock file#3134

Merged
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
clarkbw:greenkeeper-lockfile
Jun 8, 2017
Merged

Switch to use greenkeeper-lockfile-update for yarn.lock file#3134
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
clarkbw:greenkeeper-lockfile

Conversation

@clarkbw

@clarkbw clarkbw commented Jun 8, 2017

Copy link
Copy Markdown
Contributor

Associated Issue: #2978

Summary of Changes

Test Plan

This is one of those PRs where the test plan is merging it and seeing that it works as expected with future greenkeeper PRs.

But currently I believe we're seeing a number of errors happening with CircleCI and multiple attempts to update the yarn.lock file. Example: https://github.com/devtools-html/debugger.html/commits/greenkeeper/jest-localstorage-mock-1.2.0

@codecov

codecov Bot commented Jun 8, 2017

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3134   +/-   ##
=======================================
  Coverage   47.37%   47.37%           
=======================================
  Files          95       95           
  Lines        4000     4000           
  Branches      825      825           
=======================================
  Hits         1895     1895           
  Misses       2105     2105

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 d2af129...3fcc8dc. Read the comment docs.

@jasonLaster

Copy link
Copy Markdown
Contributor

Thanks! SHould i merge?

@jasonLaster jasonLaster left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Wow, this is awesome

@jasonLaster jasonLaster merged commit bccea79 into firefox-devtools:master Jun 8, 2017
@clarkbw clarkbw deleted the greenkeeper-lockfile branch June 8, 2017 21:59
@clarkbw clarkbw mentioned this pull request Jun 8, 2017
10 tasks
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.

2 participants