search

guzru / winston-sentry

Sentry transport for winston logger for node js
MIT License
82 stars 37 forks source link

Log the reason an error occurred. #29

Closed Peter-Barrett closed 8 years ago

Peter-Barrett commented 8 years ago

This is a straightforward PR, I just wanted more info on why an error was occurring.

Peter-Barrett commented 8 years ago

@shaharke are you maintaining this?

shaharke commented 8 years ago

Unfortunately I don't have much time to really maintain it. I get to merge PRs only once every few months. Any volunteers?

On Tue, May 31, 2016 at 5:10 PM Peter Barrett notifications@github.com wrote:

@shaharke https://github.com/shaharke are you maintaining this?

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/guzru/winston-sentry/pull/29#issuecomment-222700340, or mute the thread https://github.com/notifications/unsubscribe/AAnDv8egJfKTTFnnwKgIn-rwlLa_BBWEks5qHEFegaJpZM4IqZZo .

Peter-Barrett commented 8 years ago

I'm more than happy to look over and merge PR's anyway, it's a module I use pretty frequently.

shaharke commented 8 years ago

@Peter-Barrett since I don't own this repo I can't really add you as contributor. @guzru can you add him?

Peter-Barrett commented 8 years ago

No problem, let me know @guzru, cheers.

guzru commented 8 years ago

Done! Sorry for the wait :)

Peter-Barrett commented 8 years ago

No problem at all, should I bump the version to 0.1.2 for this?