you-dont-need/You-Dont-Need-Lodash-Underscore

Add more suggestions for _.first

Open

#312 aperta il 2 mag 2021

Vedi su GitHub
 (1 commento) (1 reazione) (0 assegnatari)JavaScript (811 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (18.379 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Currently, the suggestion we get for _.first might not be relevant in all cases. Array.prototype.slice() returns an array. But _.first from lodash returns a non-array value. So this line should be something like this I guess:

"alternative": "Array.prototype.slice() or arr[0]"

Also, I think we should add more aliases here. For example _.head should be prevented when _.first is prevented, because they're basically the same.

Guida contributor

Direzione di ricerca
Individua il file lib/rules/rules.json alla riga 51. Modifica il campo alternative per .first per includere 'arr[0]' in aggiunta a Array.prototype.slice(). Aggiungi anche alias come .head da prevenire quando .first è prevenuto. Controlla il pattern esistente per aggiungere alias nel file delle regole.
Tech stack
javascript
Dominio
documentation
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
basic JavaScriptJSON formatfamiliarity with lodash
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70