-
Notifications
You must be signed in to change notification settings - Fork 950
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
Warning fixes and other cleanups #62
Conversation
body, _ := ioutil.ReadFile(filename) | ||
w.Write([]byte(body)) | ||
} | ||
|
||
func TestNewClient(t *testing.T) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not delete these unused test functions. We need to start adding some tests and these helper functions will be needed when we do... So I would like to keep these in.
Hi @mvdan, looks good! Thanks! I would like to keep the (currently) unused test helper functions, so if you can revert deleting them I'll merge the rest of this PR 👍 |
@mvdan I still would like to pull this one in, but you would first have to rebase it and revert the part where you delete the unused test functions. If you can do that, that would be great and I'll merge this PR right after... Thanks! |
I just went on holidays right after opening this, so give me a couple of days :) |
Sure, no rush! Thx 😀 |
It has methods to write strings and bytes already, which is much easier and also faster than going through a regular write of []byte.
@svanharmelen done. |
No description provided.