sindresorhus/eslint-plugin-unicorn

`consistent-existence-index-check`: No rationale given for style

Open

Aperta il 6 ago 2025

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (5022 star) (468 fork)user submission
docsgood for beginnerhelp wanted

Descrizione

Description

This rule was introduced: https://github.com/sindresorhus/eslint-plugin-unicorn/issues/1024, and our codebase uses the style: >= 0 and < 0 rather than !== -1 or === -1. Why this is preferred should either be documented, or a alternative style option could be provided.

Examples

This potential option would invert the typical behavior of the rule.

const index = foo.indexOf('bar');

// ❌
if (index === -1) {}


// ✅
if (index < 0) {}
const index = foo.indexOf('bar');

// ❌
if (index !== -1) {}

// ✅
if (index >= 0) {}

Additional Info

No response

Guida contributor