Skip to content

Conversation

@lpinca
Copy link
Member

@lpinca lpinca commented Apr 1, 2021

console.trace() was used when printing the warning for a possible
event listeners leak. It is no longer used since commit c6656db that
introduced the process.emitWarning() API and used it to print the
warning.

`console.trace()` was used when printing the warning for a possible
event listeners leak. It is no longer used since commit c6656db that
introduced the `process.emitWarning()` API and used it to print the
warning.
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Apr 1, 2021
@nodejs-github-bot
Copy link
Collaborator

@marsonya marsonya added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 6, 2021
@nodejs-github-bot
Copy link
Collaborator

lpinca added a commit that referenced this pull request Apr 6, 2021
`console.trace()` was used when printing the warning for a possible
event listeners leak. It is no longer used since commit c6656db that
introduced the `process.emitWarning()` API and used it to print the
warning.

PR-URL: #38016
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@lpinca
Copy link
Member Author

lpinca commented Apr 6, 2021

Landed in d61977f.

@lpinca lpinca closed this Apr 6, 2021
@lpinca lpinca deleted the remove/dead-code branch April 6, 2021 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants