Skip to content
This repository was archived by the owner on Jan 21, 2026. It is now read-only.

feat: support tracing for untranspiled async/await in Node 8+#775

Merged
kjin merged 1 commit intogoogleapis:masterfrom
kjin:ah-default-v3
Jun 28, 2018
Merged

feat: support tracing for untranspiled async/await in Node 8+#775
kjin merged 1 commit intogoogleapis:masterfrom
kjin:ah-default-v3

Conversation

@kjin
Copy link
Copy Markdown
Contributor

@kjin kjin commented Jun 13, 2018

Fixes #566

This PR removes references to GCLOUD_TRACE_NEW_CONTEXT. Code that depend on its value now behave as if it were always true.

This change makes async_hooks-based tracing the default behavior in Node 8+.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 13, 2018
@kjin kjin force-pushed the ah-default-v3 branch 2 times, most recently from 6a752f9 to 6e97a72 Compare June 14, 2018 17:37
@kjin kjin requested a review from a team June 14, 2018 17:37
Comment thread package.json
"@types/is": "0.0.20",
"@types/methods": "^1.1.0",
"@types/mocha": "^5.0.0",
"@types/mocha": "5.2.1",

This comment was marked as spam.

Copy link
Copy Markdown
Contributor

@ofrobots ofrobots left a comment

Choose a reason for hiding this comment

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

semver minor?

@kjin
Copy link
Copy Markdown
Contributor Author

kjin commented Jun 28, 2018

@ofrobots Yes. Thanks!

@kjin kjin merged commit 30d0529 into googleapis:master Jun 28, 2018
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