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

Métriques du dépôt

Stars

5 022 étoiles (5 022 étoiles)

Métriques de merge PR

Merge moyen 1j 16h (Merge moyen 1j 16h)399 PRs mergées en 30 j (399 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche

Étudiez l'implémentation de la règle dans `rules/no useless spread.js`. Ajoutez une condition pour exclure les cas où la source est un appel `concat` qui peut produire des tableaux creux. Mettez à jour le fichier de test `test/no useless spread.mjs` avec l'exemple fourni dans l'issue. Consultez la documentation MDN sur les tableaux creux et le comportement de `concat`.

Stack technique

javascript

Domaine

tooling

Type d'issue

Bug

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 heures

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

Active

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

Claire

Prérequis

JavaScriptESLint

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

60