unicorn/prefer-includes Style

🚧 An auto-fix is planned for this rule, but not implemented at this time.

What it does

Prefer includes() over indexOf() when checking for existence or non-existence. All built-ins have .includes() in addition to .indexOf().

Why is this bad?

The .includes() method is more readable and less error-prone than .indexOf().

Examples

Examples of incorrect code for this rule:

if (str.indexOf("foo") !== -1) {}

Examples of correct code for this rule:

if (str.includes("foo")) {}

How to use

To enable this rule using the config file or in the CLI, you can use:

Config (.oxlintrc.json)

{
  "rules": {
    "unicorn/prefer-includes": "error"
  }
}

CLI

oxlint --deny unicorn/prefer-includes

References

• Rule Source