immobiliare/fastify-sentry

Improve ts types

Closed this issue · 5 comments

dnlup commented

It looks like some types are not resolved on the fastify.Sentry instance as they would when using the Sentry instance.

dnlup commented

Maybe we should re-export all the Sentry types 🤔

dnlup commented

@mdoria12 wdyt?

I've tried by adding this at the end of the index.d.ts file:
export * from '@sentry/types';

and this solve my use case of using the Severity enum.
But it seems that we should not use this approach anymore, because the enum is exported as a type and deprecated by Sentry itself, so i think we can keep the current types as is and close this issue for now.

@dnlup wdyt?

dnlup commented

I am thinking if we could have another use case similar to yours that could benefit of exporting all the types 🤔

dnlup commented

Maybe we can consider a rewrite in TS in a new release 🤔