[mongodb] use T as type rather than value

[kjin]

A recent change made T a value rather than a type. This PR fixes this.

• Use a meaningful title for the pull request. Include the name of the package modified.
• Test the change in your own code. (Compile and run.)
• Add or edit tests to reflect the change. (Run with npm test.)
• Follow the advice from the readme.
• Avoid common mistakes.
• Run npm run lint package-name (or tsc if no tslint.json is present).

[typescript-bot]

@kjin Thank you for submitting this PR!

🔔 @CaselIT @alanmarcell @kikar @bitjson @Dante-101 @mcortesi @EnricoPicci @AJCStriker @julien-c @daprahamian - please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

If no reviewer appears after a week, a DefinitelyTyped maintainer will review the PR instead.

[typescript-bot]

A definition owner has approved this PR ⭐️. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait. Thank you for your contribution to DefinitelyTyped!

[ravihugo]

I just updated to the latest version and started getting this error:
node_modules/@types/mongodb/index.d.ts(1255,24): error TS7006: Parameter 'T' implicitly has an 'any' type.

Presumably from this commit? Not sure if anyone else is seeing this.

Workaround:
npm i -D @types/mongodb@3.0.18

Edit: I originally mistakenly thought this PR was the cause - but it's actually the fix.. just waiting for @types/mongodb to get this update.

[CaselIT]

This should fix it

[adriang133]

@ravihugo Strangely, I don't get an error when I run tsc locally, but when I deploy to Heroku I get that error. Tried 3.0.18 also, but still get the same :(

[GingerLoaf]

So this error was fixed on the 15th and it is now the 18th, and I still see the latest npm version at 3.0.20 (https://www.npmjs.com/package/@types/mongodb). How long does it usually take for this to propagate to npm?

[kjin]

@GingerLoaf I think it gets published automatically after someone merges it. So hopefully, some time today.