Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor: remove deprecated util.error #699

Closed
wants to merge 2 commits into from

Conversation

mattlyons0
Copy link
Contributor

Replace with console.error
Replace util.log with console.log to be consistent with _log

Replace with console.error
Replace util.log with console.log to be consistent with _log
@mattlyons0
Copy link
Contributor Author

I can't mange to reproduce the failed build. All the tests pass for me using Node versions: (4.2.1), (0.12.7) and (0.10.29). I have been using a different npm version (3.3.8), but I don't see how that could cause this.

@mattlyons0
Copy link
Contributor Author

I think I commited this next fix minutes after you merged the original PR. Should I update my fork and make a new PR?

@remy
Copy link
Owner

remy commented Feb 25, 2016

I think these changes actually got merged in a long time ago! Sorry.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants