sindresorhus/eslint-plugin-unicorn

`no-nested-ternary` does not autofix nested ternary

Geschlossen

#2.008 geöffnet am 07.12.2022

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (468 Forks)user submission
docshelp wantedwontfix

Repository-Metriken

Stars
 (5.022 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 4h 30m) (26 gemergte PRs in 30 T)

Beschreibung

var cellText = isYear
    ? i
    : isMonth
        ? settings.text.monthsShort[i]
        : isDay
            ? cellDate.getDate()
            : module.helper.dateFormat(formatter.cellTime, cellDate);

When the rule can autofix ternary (add parenthesis around), I would expect it to fix any level of nesting.

Currently it seems it can autofix only 2 nested ternary operators.

Contributor Guide

Research-Richtung
Untersuche die Quelldatei `source/rules/no nested ternary.js`, um zu verstehen, wie die Autofix Funktion derzeit für zweistufige Verschachtelung implementiert ist. Suche nach der Fix Funktion, die Klammern hinzufügt, und bestimme, wie sie erweitert werden kann, um rekursiv mit beliebiger Tiefe verschachtelter ternärer Operatoren umzugehen. Überprüfe die vorhandenen Tests, um die Abdeckung zu sehen, und füge Testfälle für dreifache oder tiefere Verschachtelung hinzu.
Tech Stack
javascript
Domain
tooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptESLint
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70