Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Conversation

@surajnarwade
Copy link
Contributor

Fixed namechange in Sirupsen/logrus to sirupsen/logrus

@surajnarwade
Copy link
Contributor Author

cc @vdemeester

Copy link
Contributor

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

You need to rm -fR vendor/github.com/Sirupsen to make sure it checks out the logrus lib with the lowercase. Once you do that, the build will fail and you will need to change the import path to github.com/sirupsen/logrus 😉

@vdemeester
Copy link
Contributor

@surajnarwade Updated your PR (by ammending the commit). Let's hope it doesn't fail 😓

@vdemeester vdemeester force-pushed the fix-logrus branch 3 times, most recently from a13d295 to 4a4bf14 Compare October 24, 2017 16:11
Fixed namechange in `Sirupsen/logrus` to `sirupsen/logrus`

Signed-off-by: Suraj Narwade <[email protected]>
Signed-off-by: Vincent Demeester <[email protected]>
Copy link
Contributor

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🦁
It is green 🍏

@vdemeester vdemeester merged commit 57bd716 into docker-archive-public:master Oct 25, 2017
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.

3 participants