gocheck

Merge lp://qastaging/~rogpeppe/gocheck/fix-error-printing into lp://qastaging/gocheck

  1. fix-error-printing
  2. Merge into trunk
Proposed by Roger Peppe
Status: Merged
Merged at revision: 66
Proposed branch: lp://qastaging/~rogpeppe/gocheck/fix-error-printing
Merge into: lp://qastaging/gocheck
Diff against target: 58 lines (+14/-9)
2 files modified
checkers.go (+1/-1)
gocheck.go (+13/-8)
To merge this branch: bzr merge lp://qastaging/~rogpeppe/gocheck/fix-error-printing
Reviewer Review Type Date Requested Status
Gustavo Niemeyer Pending
Review via email: mp+82714@code.qastaging.launchpad.net

Description of the change

Fix errors to print their string values.

Also, fmt.Stringer is a more conventional interface
type than hasString.

https://codereview.appspot.com/5418043/

To post a comment you must log in.
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

LGTM, thanks.

https://codereview.appspot.com/5418043/

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

*** Submitted:

Fix errors to print their string values.

Also, fmt.Stringer is a more conventional interface
type than hasString.

R=
CC=
https://codereview.appspot.com/5418043

https://codereview.appspot.com/5418043/

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches