Skip to content

github.com/sirupsen/logrus -> github.com/Sirupsen/logrus#5

Closed
AkihiroSuda wants to merge 1 commit intocontainerd:masterfrom
AkihiroSuda:fix-logrus
Closed

github.com/sirupsen/logrus -> github.com/Sirupsen/logrus#5
AkihiroSuda wants to merge 1 commit intocontainerd:masterfrom
AkihiroSuda:fix-logrus

Conversation

@AkihiroSuda
Copy link
Member

For ease of vendoring go-runc in containerd.
(It had required both sirupsen/logrus and Sirupsen/logrus)
https://github.com/docker/containerd/pull/415/files#r95741905

Signed-off-by: Akihiro Suda suda.akihiro@lab.ntt.co.jp

For ease of vendoring go-runc in containerd.
(It had required both sirupsen/logrus and Sirupsen/logrus)

Signed-off-by: Akihiro Suda <suda.akihiro@lab.ntt.co.jp>
@crosbymichael
Copy link
Member

@AkihiroSuda thanks. I actually pushed a commit returning the error and removing logrus so we don't have to worry about this anymore for the package. I think its better overall for the package and solves this problem as well.

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.

2 participants