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

Update dependencies to use weaveworks/common upstream #1226

Merged
merged 1 commit into from
Nov 5, 2019

Conversation

clickyotomy
Copy link
Contributor

@clickyotomy clickyotomy commented Nov 3, 2019

What this PR does / why we need it: Updates dependencies to use the current upstream version of weaveworks/comomon.

Which issue(s) this PR fixes: Fixes #945.

Special notes for your reviewer: None.

Checklist

  • Documentation added
  • Tests updated

@CLAassistant
Copy link

CLAassistant commented Nov 3, 2019

CLA assistant check
All committers have signed the CLA.

@Sh4d1
Copy link

Sh4d1 commented Nov 3, 2019

Also fixes #759 although its already closed 😅

@ThoreKr
Copy link

ThoreKr commented Nov 4, 2019

Should fix #1051 as well and indirectly #1038.

Copy link
Contributor

@cyriltovena cyriltovena left a comment

Choose a reason for hiding this comment

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

LGTM

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.

Move weaveworks/common back to upstream
5 participants