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

Output 'noPluginName' in trace-api log messages where pluginName is undefined#958

Merged
kjin merged 2 commits intogoogleapis:masterfrom
hcharley:patch-1
Jan 23, 2019
Merged

Output 'noPluginName' in trace-api log messages where pluginName is undefined#958
kjin merged 2 commits intogoogleapis:masterfrom
hcharley:patch-1

Conversation

@hcharley
Copy link
Copy Markdown
Contributor

This will change log messages like this from:

[JS] @google-cloud/trace-agent DEBUG TraceApi#createChildSpan: [undefined] Created child span [doSomething()]
[JS] @google-cloud/trace-agent DEBUG TraceApi#createChildSpan: [undefined] Created child span [doSomething() -> browserStart]

To this:

[JS] @google-cloud/trace-agent DEBUG TraceApi#createChildSpan: [noPluginName] Created child span [doSomething()]
[JS] @google-cloud/trace-agent DEBUG TraceApi#createChildSpan: [noPluginName] Created child span [doSomething() -> browserStart]

… undefined

This will change log messages like this from:

```
[JS] @google-cloud/trace-agent DEBUG TraceApi#createChildSpan: [undefined] Created child span [doSomething()]
[JS] @google-cloud/trace-agent DEBUG TraceApi#createChildSpan: [undefined] Created child span [doSomething() -> browserStart]
```

To this:

```
[JS] @google-cloud/trace-agent DEBUG TraceApi#createChildSpan: [noPluginName] Created child span [doSomething()]
[JS] @google-cloud/trace-agent DEBUG TraceApi#createChildSpan: [noPluginName] Created child span [doSomething() -> browserStart]
```
@googlebot
Copy link
Copy Markdown

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Jan 19, 2019
@hcharley
Copy link
Copy Markdown
Contributor Author

I signed it!

Although, I did get this error when I attempted to sign it:

screen shot 2019-01-19 at 2 39 35 am

@googlebot
Copy link
Copy Markdown

CLAs look good, thanks!

@googlebot googlebot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Jan 19, 2019
Comment thread src/trace-api.ts Outdated
Copy link
Copy Markdown
Contributor

@kjin kjin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@kjin kjin merged commit 6793b09 into googleapis:master Jan 23, 2019
@hcharley
Copy link
Copy Markdown
Contributor Author

Thank you @kjin!

@hcharley hcharley deleted the patch-1 branch January 29, 2019 16:53
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