Skip to content

logging-bunyan: default removeCircular to true#2094

Merged
stephenplusplus merged 1 commit intogoogleapis:masterfrom
ofrobots:bunyan-winston-circular
Mar 16, 2017
Merged

logging-bunyan: default removeCircular to true#2094
stephenplusplus merged 1 commit intogoogleapis:masterfrom
ofrobots:bunyan-winston-circular

Conversation

@ofrobots
Copy link
Contributor

Fixes: #1987

winston doesn't have this issue as objects will get formatted with util.inspect which will naturally removes circularity.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 16, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.924% when pulling b1bcf78 on ofrobots:bunyan-winston-circular into 402c534 on GoogleCloudPlatform:master.

@stephenplusplus stephenplusplus merged commit 70682b1 into googleapis:master Mar 16, 2017
sofisl pushed a commit that referenced this pull request Feb 25, 2026
#2094)

* fix: remove async from final function which causes double invocation of callback

* add catch clause
sofisl pushed a commit that referenced this pull request Feb 26, 2026
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [google-gax](https://togithub.com/googleapis/gax-nodejs) ([source](https://togithub.com/googleapis/gax-nodejs/tree/HEAD/gax)) | [`4.3.8` -> `4.3.9`](https://renovatebot.com/diffs/npm/google-gax/4.3.8/4.3.9) | [![age](https://developer.mend.io/api/mc/badges/age/npm/google-gax/4.3.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/google-gax/4.3.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/google-gax/4.3.8/4.3.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/google-gax/4.3.8/4.3.9?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/gax-nodejs (google-gax)</summary>

### [`v4.3.9`](https://togithub.com/googleapis/gax-nodejs/blob/HEAD/gax/CHANGELOG.md#439-2024-08-06)

[Compare Source](https://togithub.com/googleapis/gax-nodejs/compare/google-gax-v4.3.8...google-gax-v4.3.9)

##### Bug Fixes

-   reduce duplicate code in streaming retries and add a test ([#&#8203;1636](https://togithub.com/googleapis/gax-nodejs/issues/1636)) ([83b52e1](https://togithub.com/googleapis/gax-nodejs/commit/83b52e14ba3bb1e9f0e079ea53e47ec32b4cfd45))
-   test application cleanup ([#&#8203;1639](https://togithub.com/googleapis/gax-nodejs/issues/1639)) ([76ac496](https://togithub.com/googleapis/gax-nodejs/commit/76ac496fe42f1176df733d0adcc755bd4e773b10))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 9am and before 3pm" (UTC), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR was generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View the [repository job log](https://developer.mend.io/github/googleapis/nodejs-spanner).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4yNi4xIiwidXBkYXRlZEluVmVyIjoiMzguMjYuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->
sofisl pushed a commit that referenced this pull request Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants