Skip to content
New issue

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

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

Already on GitHub? # to your account

docs->added pino-hana transport #2059

Merged
merged 1 commit into from
Oct 12, 2024
Merged

Conversation

HiImGiovi
Copy link
Contributor

Hi,

I wrote pino-hana a pino v7+ transport which writes logs into SAP HANA database.
I added its reference in the docs of the official pino page.

Also can you please review the implementation of pino-hana if possible?
I'd really like to have an opinion about potential issues with the implementation!

Thank you!

Simone

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@jsumners jsumners merged commit d32fd55 into pinojs:main Oct 12, 2024
alexandresoro pushed a commit to alexandresoro/ouca that referenced this pull request Oct 16, 2024
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [pino](https://getpino.io) ([source](https://github.com/pinojs/pino)) | dependencies | minor | [`9.4.0` -> `9.5.0`](https://renovatebot.com/diffs/npm/pino/9.4.0/9.5.0) |

---

### Release Notes

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

### [`v9.5.0`](https://github.com/pinojs/pino/releases/tag/v9.5.0)

[Compare Source](pinojs/pino@v9.4.0...v9.5.0)

#### What's Changed

-   build(deps-dev): bump [@&#8203;yao-pkg/pkg](https://github.com/yao-pkg/pkg) from 5.12.1 to 5.13.0 by [@&#8203;dependabot](https://github.com/dependabot) in pinojs/pino#2042
-   build(deps): bump pino-abstract-transport from 1.2.0 to 2.0.0 by [@&#8203;dependabot](https://github.com/dependabot) in pinojs/pino#2043
-   Update transports.md by [@&#8203;Jhon-Mosk](https://github.com/Jhon-Mosk) in pinojs/pino#2041
-   doc: Add Transport [@&#8203;macfja/pino-fingers-crossed](https://github.com/macfja/pino-fingers-crossed) by [@&#8203;MacFJA](https://github.com/MacFJA) in pinojs/pino#2047
-   build(deps-dev): bump [@&#8203;yao-pkg/pkg](https://github.com/yao-pkg/pkg) from 5.13.0 to 5.15.0 by [@&#8203;dependabot](https://github.com/dependabot) in pinojs/pino#2050
-   docs->added pino-hana transport by [@&#8203;HiImGiovi](https://github.com/HiImGiovi) in pinojs/pino#2059
-   docs -> [#&#8203;2061](pinojs/pino#2061) Alphabetize transport list and reorder sections in documentation by [@&#8203;superherobtf1985](https://github.com/superherobtf1985) in pinojs/pino#2062
-   feature: allow sync flag to be passed to ThreadStream for synchronous… by [@&#8203;karankraina](https://github.com/karankraina) in pinojs/pino#2063
-   fix: Ensure silent log level is respected with browser.transmit option by [@&#8203;SrsDanny](https://github.com/SrsDanny) in pinojs/pino#2052

#### New Contributors

-   [@&#8203;MacFJA](https://github.com/MacFJA) made their first contribution in pinojs/pino#2047
-   [@&#8203;HiImGiovi](https://github.com/HiImGiovi) made their first contribution in pinojs/pino#2059
-   [@&#8203;superherobtf1985](https://github.com/superherobtf1985) made their first contribution in pinojs/pino#2062
-   [@&#8203;karankraina](https://github.com/karankraina) made their first contribution in pinojs/pino#2063
-   [@&#8203;SrsDanny](https://github.com/SrsDanny) made their first contribution in pinojs/pino#2052

**Full Changelog**: pinojs/pino@v9.4.0...v9.5.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/226
Reviewed-by: Alexandre Soro <code@soro.dev>
Co-authored-by: renovate <renovate@git.tristess.app>
Co-committed-by: renovate <renovate@git.tristess.app>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants