Skip to content
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

fix: Remove the process warning listener#376

Merged
DominicKramer merged 1 commit intogoogleapis:masterfrom
DominicKramer:fix/remove-process-on-warning-listener
Dec 12, 2017
Merged

fix: Remove the process warning listener#376
DominicKramer merged 1 commit intogoogleapis:masterfrom
DominicKramer:fix/remove-process-on-warning-listener

Conversation

@DominicKramer
Copy link
Copy Markdown
Contributor

Node 8.5.0 introduced a change where a warning message discussing
the use of async traces would be printed when using the debug
agent with the inspector protocol. At that time, a listener was
added that listened to that warning and also printed a message
stating that the warning does not impact the debug agent.

At this point, the underlying issue causing the warning message to
appear has been fixed in Node, and, as such, the associated warning
listener in the debug agent has be removed.

Node 8.5.0 introduced a change where a warning message discussing
the use of async traces would be printed when using the debug
agent with the inspector protocol.  At that time, a listener was
added that listened to that warning and also printed a message
stating that the warning does not impact the debug agent.

At this point, the underlying issue causing the warning message to
appear has been fixed in Node, and, as such, the associated warning
listener in the debug agent has be removed.
@DominicKramer DominicKramer requested a review from a team December 11, 2017 23:30
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants