fixed index.d.ts

[spb-web]

Type 'typeof TransportStream' is not a constructor function type.

[schfkt]

@spb-web do you have esModuleInterop compiler option set to true in your tsconfig?

The thing is, you commit fixes the typings for the case when that option is set to true, but breaks it for the cases when that option is disabled. See yourself with this sample code: https://github.com/schfkt/graylog-test

So we need to find a way to make it work for the both cases when esModuleInterop is enabled and disabled.

[julioocamargoo]

I'm facing the same problem. Maybe a major version will be needed. esModuleInterop is a default config on TS 3.0 and seems like the community is heading this way.

[glensc]

The same problem reported as an issue #82

[glensc]

This PR is not needed, just import properly:

• ERROR:Type 'typeof TransportStream' is not a constructor function type. Version: typescript 3.5.3, #82 (comment)

Sample was taken from example given by @schfkt:

• https://github.com/schfkt/graylog-test/blob/7491d0c3897657738d059ddb1ea690ec8822e100/src/index.ts#L2

[scalder27]

I think this PR is needed - it's not okay when a library tells you what ts settings you should use. But as @schfkt said - we need a solution that equally works with esModuleInterop's both values.

How about this one?

import TransportStream = require('winston-transport');

[schfkt]

@scalder27 Thanks for the suggestion. It works for both cases: when esModuleInterop is enabled or disabled. Here's the PR with the fix: #84

[schfkt]

Closing this in favor of #84