Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
package_test.go: tiny fix to panic output #16
Conversation
davecheney
reviewed
Feb 6, 2015
| @@ -82,7 +82,7 @@ func setLocationsForErrorTags(filename string) { | ||
| if j := strings.Index(line, "//err "); j >= 0 { | ||
| tag := line[j+len("//err "):] | ||
| if _, found := tagToLocation[tag]; found { | ||
| - panic(fmt.Errorf("tag %q already processed previously")) | ||
| + panic(fmt.Errorf("tag %q already processed previously", tag)) |
|
@davecheney I assume you meant Sprintf |
|
yup On Fri, Feb 6, 2015 at 8:41 PM, Matt Williams notifications@github.com
|
|
LGTM |
|
$$merge$$ |
added a commit
that referenced
this pull request
Feb 11, 2015
fwereade
merged commit 036046b
into
juju:master
Feb 11, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
mattyw commentedFeb 6, 2015
No description provided.