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

Description

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

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


// ✅
if (index < 0) {}

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

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

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

Guide contributeur

Stack technique

javascriptnodejs

Domaine

developer experiencedocumentation

Type d'issue

feature

DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.

2

Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.

1-3 hours

Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.

fresh

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

clear

Prérequis

basic JavaScriptESLint fundamentals

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

80

Direction de recherche

Review the implementation of the `consistent existence index check` rule in the plugin source code (likely `rules/consistent existence index check.js`). Examine the rationale behind the current style (`>= 0`/`< 0` vs `!== 1`/`=== 1`). Consider adding documentation to the rule's documentation or adding a new option to allow the alternative style. Check the referenced issue #1024 for prior discussion on the rationale.