aandrewww / pino-sentry

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

Sentry should not be a dependency but rather a peer dependency #57

Open mwillbanks opened 1 year ago

mwillbanks commented 1 year ago

Currently sentry is a dependency of this package, however, it should become a peer dependency since people may already have a package installed.

glensc commented 1 month ago

This won't work the way this package is supposed to be installed: globally:

npm install pino-sentry -g

how do you handle peer dependencies there?

See install:

And cli usage: