sindresorhus/eslint-plugin-unicorn

Rule proposal: `prefer-unshift`

Chiusa

#2165 aperta il 6 lug 2023

Vedi su GitHubGuida contributor
 (2 commenti) (2 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
help wantednew rule

Metriche repository

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

Descrizione

Description

I see my newbine team member write following code when I code review:

 sItems.splice(0, 0, { itemType: 'Ungroup' });

Fail

arr.splice(0, 0, {})

Pass

arr.unshift({})

Additional Info

maybe another proposal to suggest:

arr.splice(0, 1)

to:

arr.shift()

Guida contributor

Direzione di ricerca
Implementare una nuova regola ESLint chiamata `prefer unshift` che rileva `Array.prototype.splice(0, 0, ...)` e suggerisce `unshift`, e `splice(0, 1)` suggerisce `shift`. Studiare le regole esistenti in questo plugin per i pattern.
Tech stack
javascript
Dominio
tooling
Tipo issue
Funzionalità
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
ESLint plugin development
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75