Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
api/block: driveby cleanups #4401
Conversation
mjs
reviewed
Feb 26, 2016
| - return result.Error | ||
| - } | ||
| - return nil | ||
| + return errors.Trace(result.Error) |
|
LGTM |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$JFDI$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$JFDI$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$JFDI$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
added a commit
that referenced
this pull request
Mar 7, 2016
davecheney commentedFeb 12, 2016
Multi line logging, seriously ??