eslint config: "@typescript-eslint/no-floating-promises": 2

[naseemkullah]

Thoughts on adding this rule?

This rule forbids usage of Promise-like values in statements without handling their errors appropriately. Unhandled promises can cause several issues, such as improperly sequenced operations, ignored Promise rejections and more. Valid ways of handling a Promise-valued statement include awaiting, returning, and either calling .then() with two arguments or .catch() with one argument.

• This only affects the JavaScript OpenTelemetry library
• This may affect other libraries, but I would like to get opinions here first

[dyladan]

While you're updating the eslint rules you may want to take a look at the current warns and determine if any of those rules should be disabled or have options configured. For instance, I know the following code will fail:

// unused variable a
function log(a: unknown, b: string) {
  console.log(b);
}

In this case we might want this to fail, but it may also be implementing some interface but just doesn't care about the first argument?

[pichlermarc]

Done in #1780 🙂