`no-useless-spread` it is not safe to remove spread in `[...foo.concat(bar)]`

Metriche repository

Star

5022 stelle (5022 stelle)

Metriche merge PR

Merge medio 1g 16h (Merge medio 1g 16h)399 PR mergiate in 30 g (399 PR mergiate in 30 g)

Descrizione

const foo = new Array(5);
const bar = new Array(5);

const cat = [...foo.concat(bar)];
// now we got a new array with undefined x 10
// but foo.concat(bar) is empty x 10.
// It's not what we want.

Guida contributor

Direzione di ricerca

Analizza l'implementazione della regola in `rules/no useless spread.js`. Aggiungi una condizione per escludere i casi in cui la sorgente è una chiamata `concat` che potrebbe produrre array sparsi. Aggiorna il file di test `test/no useless spread.mjs` con l'esempio fornito nell'issue. Rivedi la documentazione MDN sugli array sparsi e il comportamento di `concat`.

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.

Attiva

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Chiara

Prerequisiti

JavaScriptESLint

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

60