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

feat: Replace console.log with debug #155

Merged
merged 1 commit into from
Jan 3, 2016
Merged

feat: Replace console.log with debug #155

merged 1 commit into from
Jan 3, 2016

Conversation

dignifiedquire
Copy link
Member

This is the setup for #151. The actual good logging messages still need to be written at a later point

@dignifiedquire
Copy link
Member Author

After some discussion on IRC we might go with debug needs more research

@dignifiedquire dignifiedquire changed the title feat: Replace console.log and debug with bows feat: Replace console.log with debug Jan 3, 2016
@dignifiedquire
Copy link
Member Author

Changed to debug as that seems to be the choice these days and it supports stderr in node.

@dignifiedquire dignifiedquire mentioned this pull request Jan 3, 2016
@dignifiedquire
Copy link
Member Author

@RichardLitt can you review this one as well and merge if you are happy?

@RichardLitt
Copy link
Member

Everybody wins.

RichardLitt added a commit that referenced this pull request Jan 3, 2016
feat: Replace console.log with debug
@RichardLitt RichardLitt merged commit 9327e46 into master Jan 3, 2016
@RichardLitt RichardLitt deleted the bows branch January 3, 2016 20:52
@jbenet jbenet removed the codereview label Jan 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants