sindresorhus/eslint-plugin-unicorn

`no-useless-fallback-in-spread`: Support ternary

Chiusa

#1861 aperta il 19 lug 2022

Vedi su GitHubGuida contributor
 (4 commenti) (7 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
enhancementhelp wanted

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 4h 30m) (26 PR mergiate in 30 g)

Descrizione

The rule currently does not detect ternary like this:

{...(foo ? {bar: true} : {})}

This can be safely be reduced to

{...(foo && {bar: true})}

Guida contributor

Direzione di ricerca
Il problema consiste nell'estendere la regola `no useless fallback in spread` per rilevare espressioni ternarie come `{...(foo ? {bar: true} : {})}` e suggerire l'uso di `{...(foo && {bar: true})}`. Inizia esaminando l'implementazione esistente della regola in `rules/no useless fallback in spread.js` e i relativi test. Cerca come gestisce attualmente i diversi pattern. La correzione probabilmente comporta l'aggiunta di un controllo per i nodi `ConditionalExpression` e l'assicurarsi che il conseguente sia un'espressione oggetto e l'alternativa sia un oggetto vuoto. Fai riferimento alla guida delle regole ESLint per i tipi di nodi AST. Implementa la logica e aggiungi i corrispondenti casi di test.
Tech stack
javascript
Dominio
toolingdeveloper experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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
Understanding of ASTFamiliarity with ESLint rule structureBasic JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75