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 github.com/maybebtc/logrus dependency #385

Merged
merged 1 commit into from
Nov 25, 2014

Conversation

chriscool
Copy link
Contributor

This patch was generated by running:

$ godep update github.com/maybebtc/logrus

Without this, godep restore fails like this:

$ LANG=C godep restore
fatal: reference is not a tree: 005463f8fb49c4a6f36181e05157e111ec5cf8a3
godep: restore: exit status 128

@maybebtc you might want to have a look, thanks!

This patch was generated by running:

$ godep update github.com/maybebtc/logrus

Without this, `godep restore` fails like this:

```
$ LANG=C godep restore
fatal: reference is not a tree: 005463f8fb49c4a6f36181e05157e111ec5cf8a3
godep: restore: exit status 128
```

License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@btc
Copy link
Contributor

btc commented Nov 25, 2014

my mistake. 005463 refers to an unpublished, unnecessary commit. That commit introduced the custom JSON formatter. Instead of including the formatter in logrus, we include it in eventlog.

This is TRTTD. Thanks @chriscool

LGTM

btc pushed a commit that referenced this pull request Nov 25, 2014
Update github.com/maybebtc/logrus dependency
@btc btc merged commit 1820216 into ipfs:master Nov 25, 2014
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