Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wait for destination to close #533

Merged
merged 6 commits into from
Oct 15, 2024
Merged

Wait for destination to close #533

merged 6 commits into from
Oct 15, 2024

Conversation

mcollina
Copy link
Member

Signed-off-by: Matteo Collina <hello@matteocollina.com>
Signed-off-by: Matteo Collina <hello@matteocollina.com>
Signed-off-by: Matteo Collina <hello@matteocollina.com>
@mcollina mcollina requested a review from jsumners October 15, 2024 14:15
@mcollina mcollina changed the title Wait for desintation to close Wait for destination to close Oct 15, 2024
Signed-off-by: Matteo Collina <hello@matteocollina.com>
This reverts commit 01d413d.
This reverts commit a2738a5.

Signed-off-by: Matteo Collina <hello@matteocollina.com>
@mcollina mcollina merged commit 70ade71 into master Oct 15, 2024
30 checks passed
@mcollina mcollina deleted the remove-pump-readable-stream branch October 15, 2024 15:24
alexandresoro pushed a commit to alexandresoro/ouca that referenced this pull request Oct 15, 2024
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [pino-pretty](https://github.com/pinojs/pino-pretty) | dependencies | minor | [`11.2.2` -> `11.3.0`](https://renovatebot.com/diffs/npm/pino-pretty/11.2.2/11.3.0) |

---

### Release Notes

<details>
<summary>pinojs/pino-pretty (pino-pretty)</summary>

### [`v11.3.0`](https://github.com/pinojs/pino-pretty/releases/tag/v11.3.0)

[Compare Source](pinojs/pino-pretty@v11.2.2...v11.3.0)

#### What's Changed

-   Bump [@&#8203;types/node](https://github.com/types/node) from 20.14.13 to 22.0.0 by [@&#8203;dependabot](https://github.com/dependabot) in pinojs/pino-pretty#523
-   fix: ndjson incorrect url by [@&#8203;Livour](https://github.com/Livour) in pinojs/pino-pretty#528
-   Bump pino-abstract-transport from 1.2.0 to 2.0.0 by [@&#8203;dependabot](https://github.com/dependabot) in pinojs/pino-pretty#531
-   Bump [@&#8203;arethetypeswrong/cli](https://github.com/arethetypeswrong/cli) from 0.15.4 to 0.16.1 by [@&#8203;dependabot](https://github.com/dependabot) in pinojs/pino-pretty#530
-   Wait for destination to close by [@&#8203;mcollina](https://github.com/mcollina) in pinojs/pino-pretty#533

#### New Contributors

-   [@&#8203;Livour](https://github.com/Livour) made their first contribution in pinojs/pino-pretty#528

**Full Changelog**: pinojs/pino-pretty@v11.2.2...v11.3.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

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

♻ **Rebasing**: Whenever PR becomes conflicted, 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 has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4xMjQuMCIsInVwZGF0ZWRJblZlciI6IjM4LjEyNC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJkZXBlbmRlbmNpZXMiXX0=-->

Reviewed-on: https://git.tristess.app/alexandresoro/ouca/pulls/227
Reviewed-by: Alexandre Soro <code@soro.dev>
Co-authored-by: renovate <renovate@git.tristess.app>
Co-committed-by: renovate <renovate@git.tristess.app>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pino fails to flush on process exit when lots of data
2 participants