sindresorhus/eslint-plugin-unicorn

More powerful `new-for-builtins`

Fechada

#1.835 aberto em 27 de mai. de 2022

Ver no GitHubGuia do colaborador
 (5 comentários) (2 reações) (0 responsável)JavaScript (468 forks)user submission
enhancementhelp wanted

Métricas do repositório

Stars
 (5.022 estrelas)
Métricas de merge de PR
 (Mesclagem média 4h 30m) (26 fundiu PRs em 30d)

Description

Currently, new-for-builtins enforce builtins used either call or new.

My idea is to extend this rule to check all builtins.

  • Objects like JSON/Math/Reflect/Temporal should be neither call nor new
  • Objects from global objects like Intl.DateTimeFormat/Temporal.Now... should also be checked
  • Only fix cases new and call result the same Function('')/new Function(''), Error('')/new Error(''), use suggestion for other cases like new Symbol()(runtime error)

Guia do colaborador

Direção de pesquisa
Examine a implementação existente da regra `new for builtins` no arquivo de origem (provavelmente `rules/new for builtins.js`) para entender sua lógica atual de detecção de built ins. Em seguida, estenda a para também verificar objetos como `JSON`, `Math`, `Reflect`, `Temporal` e métodos de objetos globais como `Intl.DateTimeFormat`. Certifique se de que a regra sugira correções apropriadas para casos em que `new` e `call` são equivalentes e evite falsos positivos.
Pilha de tecnologia
javascriptnodejs
Domain
developer experiencetooling
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ESLintJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60