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

[WIP] Fix deps #661

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[WIP] Fix deps #661

wants to merge 5 commits into from

Conversation

c0ze
Copy link
Contributor

@c0ze c0ze commented Nov 28, 2017

What this PR does :

  • s/Sirupsen/sirupsen/ (embrace lower case dep convention)
  • use my fork of iron-io/runner (which also uses lowercase sirupsen import, and some other fixes) When iron-io runner is updated, we can switch back to iron-io/runner
  • remove "github.com/moby/moby/cli/config/configfile" dependency and bring it into repository (that package does not exist anymore, also I couldn't find any other suitable replacement)
  • fix github.com/ucirello/supervisor import as cirello.io/supervisor (we may want to fork this)

Previously we were on a hardcoded magic Gopkg.toml which was outdated (giving warning because of the obsolete dependencies entries) We can now do a clean dep init and dep ensure.

@c0ze c0ze changed the title Fix deps [WIP] Fix deps Nov 30, 2017
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.

1 participant