noFloatingPromises, typescript-eslint/no-floating-promises

[jpike88]

Description

https://typescript-eslint.io/rules/no-floating-promises

important would be have the ignoreVoid enabled, as adding void before a Promise can explicitly define that Promise as one that is not designed to be awaited and blocking the function's execution.

[jpike88]

this rule is critical for our projects, it prevents some very nasty situations from occurring, as floating promises can cause sync issues with our data model.

[mphill]

This would definitely help catch weird runtime errors. Looking forward to it’s support!

[denbezrukov]

Unfortunately the rule implementation requires information about types.
I'm afraid that we can't implement it fully without a compiler.

[jpike88]

I understand. This rule is actually so critical that I think I'm going to bring back eslint into our CI tests just for that rule alone.

[jpike88]

I actually think that this would be the answer, not something a linter should handle as it fundamentally can change the behaviour or the code. Don't really know if it's a linter's place to be reaching that far.

microsoft/TypeScript#13376

Can close this if you like

[public]

This rule is very important for us too and it is also 2-3x slower than any other rule we have enabled in eslint. I am happy to use a mix of eslint and rome as needed but a Rust implementation of this rule seems like it would have a significant impact to our lint times.