Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update logrus to the latest so we can use ParseLevel() for PR #8335#8932

Merged
tiborvass merged 1 commit into
moby:masterfrom
duglin:UpdateLogrus
Nov 3, 2014
Merged

Update logrus to the latest so we can use ParseLevel() for PR #8335#8932
tiborvass merged 1 commit into
moby:masterfrom
duglin:UpdateLogrus

Conversation

@duglin

@duglin duglin commented Nov 3, 2014

Copy link
Copy Markdown
Contributor

Signed-off-by: Doug Davis [email protected]

@LK4D4

LK4D4 commented Nov 3, 2014

Copy link
Copy Markdown
Contributor

Hmm, I liked tag. Maybe we can ask @sirupsen to tag new version soon?

@duglin

duglin commented Nov 3, 2014

Copy link
Copy Markdown
Contributor Author

yea I looked for a tag but there wasn't a newer one.

@sirupsen

sirupsen commented Nov 3, 2014

Copy link
Copy Markdown
Contributor

I'll tag a new one, must've forgotten

@jessfraz

jessfraz commented Nov 3, 2014

Copy link
Copy Markdown
Contributor

✋ high five OSS FTW

@sirupsen

sirupsen commented Nov 3, 2014

Copy link
Copy Markdown
Contributor

v0.6.0 tagged, see sirupsen/logrus@v0.5.1...master

@duglin

duglin commented Nov 3, 2014

Copy link
Copy Markdown
Contributor Author

Excellent - thanks @sirupsen

@jessfraz

jessfraz commented Nov 3, 2014

Copy link
Copy Markdown
Contributor

LGTM

@duglin

duglin commented Nov 3, 2014

Copy link
Copy Markdown
Contributor Author

Just updated the PR with the v0.6.0 tag.

@crosbymichael

Copy link
Copy Markdown
Contributor

@tiborvass @tianon for hack

@crosbymichael

Copy link
Copy Markdown
Contributor

LGTM

1 similar comment
@tiborvass

Copy link
Copy Markdown
Contributor

LGTM

tiborvass added a commit that referenced this pull request Nov 3, 2014
Update logrus to the latest so we can use ParseLevel() for PR #8335
@tiborvass tiborvass merged commit 38d3888 into moby:master Nov 3, 2014
@duglin duglin deleted the UpdateLogrus branch November 21, 2014 20:19
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.

6 participants