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

Add more suggestions for _.first

Open

#312 aberto em 2 de mai. de 2021

Ver no GitHub
 (1 comment) (1 reaction) (0 assignees)JavaScript (811 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (18.379 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Localize o arquivo lib/rules/rules.json na linha 51. Modifique o campo alternative para .first para incluir 'arr[0]' além de Array.prototype.slice(). Adicione também aliases como .head para serem prevenidos quando .first for prevenido. Verifique o padrão existente para adicionar aliases no arquivo de regras.
Pilha de tecnologia
javascript
Domain
documentation
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
basic JavaScriptJSON formatfamiliarity with lodash
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70