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

customizable Print* functions output #6

Merged
merged 1 commit into from Jan 9, 2015
Merged

Conversation

@walf443
Copy link
Contributor

walf443 commented Jan 9, 2015

I implemented a patch related #4.

How about is it?

…for customizing Print* function's output.
"testing"
)

type TestWriter struct {

This comment has been minimized.

Copy link
@k0kubun

k0kubun Jan 9, 2015

Owner

Is this used?

This comment has been minimized.

Copy link
@walf443

walf443 Jan 9, 2015

Author Contributor

oh. sorry. it's does not need.

@k0kubun

This comment has been minimized.

Copy link
Owner

k0kubun commented Jan 9, 2015

LGTM 👍
Thanks!

k0kubun added a commit that referenced this pull request Jan 9, 2015
customizable Print* functions output
@k0kubun k0kubun merged commit 22ddd6f into k0kubun:master Jan 9, 2015
1 check passed
1 check passed
continuous-integration/wercker Build finished
Details
func init() {
if os.Getenv("DEBUG") == "" {
pp.SetDefaultOutput(os.DevNull)

This comment has been minimized.

Copy link
@k0kubun

k0kubun Jan 9, 2015

Owner

By the way, os.DevNull can't be used as io.Writer because it is a string ("/dev/null").
I've just known that ioutil package has Discard variable and it may be proper as output suppression. 009142f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.