Skip to content

Conversation

Trott
Copy link
Member

@Trott Trott commented Sep 26, 2015

common.debug() is just util.debug() and emits a deprecation notice. Per docs, use console.error() instead.

@Trott Trott added smoketest test Issues and PRs related to the tests. and removed smoketest labels Sep 26, 2015
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this line can simply be removed

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@targos I think you're right. Done!

@Trott
Copy link
Member Author

Trott commented Sep 27, 2015

@targos
Copy link
Member

targos commented Sep 28, 2015

LGTM

common.debug() is just util.debug() and emits a deprecation notice. Per
docs, use console.error() instead.

PR-URL: nodejs#3083
Reviewed-By: Michaël Zasso <[email protected]>
Trott added a commit that referenced this pull request Sep 28, 2015
common.debug() is just util.debug() and emits a deprecation notice. Per
docs, use console.error() instead.

PR-URL: #3082
Reviewed-By: Michaël Zasso <[email protected]>
@Trott
Copy link
Member Author

Trott commented Sep 28, 2015

Landed in 44efd66

@Trott Trott closed this Sep 28, 2015
Trott added a commit that referenced this pull request Sep 30, 2015
common.debug() is just util.debug() and emits a deprecation notice. Per
docs, use console.error() instead.

PR-URL: #3082
Reviewed-By: Michaël Zasso <[email protected]>
This was referenced Sep 30, 2015
Trott added a commit that referenced this pull request Sep 30, 2015
common.debug() is just util.debug() and emits a deprecation notice. Per
docs, use console.error() instead.

PR-URL: #3082
Reviewed-By: Michaël Zasso <[email protected]>
@Trott Trott deleted the rm-common-debug branch January 13, 2022 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants