feat(types): add missing typescript definitions for reporters

[regevbr]

Fix #92

[regevbr]

@atinux @pi0 can you please CR this?

[regevbr]

There are 3 drawback here

1. I added typings for the log levels. The silent level is set to be Infinity in the code. But typescript doesn't provide a valid way to use it a type literal - Allow Inifinity and -Infinity as number literal types microsoft/TypeScript#32277. To overcome it I set it to be a random really high number.
   Since the value of infinity is not really used by the code, and only used to denote that if someone inputs a really high level, it will effectively not log anything, I think my solution will work, but I believe it will be better to set a reasonable high number (such as 9999) to be marked as the silent log level.

2. I see that you expose reporters based on env (node/browser) as far as I can figure, there is no proper way to type it...

3. In order to provide proper typings for the winston reporter, we must import it. But if someone doesn't use it, it will produce a compilation error. This error can be prevented by setting the skipLibCheck flag to true in tsconfig.json
   I usually use that flag any ways, but I guess some people will discourage it. I don't mind making the logger type there to be any if you prefer.

"skipLibCheck": true,

What do you think?

[pi0]

Can we instead use an enum?

[regevbr]

Enums won't work well on d.ts alone as you must expose a real object to represent them. I will see if I can add it to the js and then expose it as an Enum here

[regevbr]

Fixed it, can you please CR it again?

[pi0]

Hi @regevbr thanks for the PR. I think based on the amount of new types it is better to rewrite lib to typescript to auto-generate them.

[regevbr]

Hi rewriting will take a long time and I'm no available for it right now as I'm working on other projects. Do tou think we can use those typings for now?

[pi0]

Of course, we can do it until rewrite. I'm just worried about winston and log level types. Can we, for now, omit Winston and use an enum (via js) for levels?

[regevbr]

Great!
Sure already implement the enum and I will drop winston

[regevbr]

Thanks @pi0! when can we expect a new release with the changes?

[pi0]

@regevbr sorry for the delay. will be released in a few minutes. Thanks again for PR ❤️ :)

[regevbr]

Sure thing! thanks :-)

[atinux]

It's out: https://github.com/nuxt-contrib/consola/releases/tag/v2.12.0

Thank you @regevbr :)

[regevbr]

Thanks @atinux