search

aandrewww / pino-sentry

🚥 Load pino logs into Sentry
MIT License
61 stars 29 forks source link

Upgrade to @sentry/node@7.1.1 #47

Closed segevfiner closed 2 years ago

segevfiner commented 2 years ago

Upgrade to @sentry/node@7.1.1

aandrewww commented 2 years ago

Thanks for the PR I don't have free time right now, but I'll review it in a few weeks.

malikdoksoz commented 2 years ago

Upgraded to @sentry/node@7.2.0

glensc commented 2 years ago

Why the duplicate pull request?

There's already one created:

segevfiner commented 2 years ago

I wasn't sure what's the status of it since it was marked as draft. If yours is better than it should be preferred and this one closed.

glensc commented 2 years ago

it's written in the comments why it's in the draft.

segevfiner commented 2 years ago

Yeah, I noticed now.

glensc commented 2 years ago

@segevfiner can you validate https://github.com/aandrewww/pino-sentry/pull/43 is working for you as well?

segevfiner commented 2 years ago

@glensc It does. 👍