sindresorhus/eslint-plugin-unicorn

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

Chiusa

#2008 aperta il 7 dic 2022

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
docshelp wantedwontfix

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 1g 16h) (399 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Analizza il file sorgente `source/rules/no nested ternary.js` per capire come è attualmente implementata la correzione automatica per la nidificazione a due livelli. Cerca la funzione di correzione che aggiunge parentesi e determina come estenderla per gestire ricorsivamente qualsiasi profondità di operatori ternari nidificati. Controlla i test esistenti per vedere la copertura e aggiungi casi di test per nidificazioni triple o superiori.
Tech stack
javascript
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70