`no-useless-fallback-in-spread`: Support ternary

Métricas do repositório

Stars

5.022 estrelas (5.022 estrelas)

Métricas de merge de PR

Mesclagem média 4h 30m (Mesclagem média 4h 30m)26 fundiu PRs em 30d (26 fundiu PRs em 30d)

Description

{...(foo ? {bar: true} : {})}

{...(foo && {bar: true})}

Guia do colaborador

Direção de pesquisa

A questão é estender a regra `no useless fallback in spread` para detectar expressões ternárias como `{...(foo ? {bar: true} : {})}` e sugerir o uso de `{...(foo && {bar: true})}`. Comece examinando a implementação existente da regra em `rules/no useless fallback in spread.js` e seus testes. Veja como ela atualmente lida com diferentes padrões. A correção provavelmente envolve adicionar uma verificação para nós `ConditionalExpression` e garantir que o consequente seja uma expressão de objeto e a alternativa seja um objeto vazio. Consulte o guia de regras do ESLint para tipos de nós AST. Implemente a lógica e adicione os casos de teste correspondentes.

Pilha de tecnologia

javascript

Domain

toolingdeveloper experience

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-3 horas

Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.

Recente

ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.

Claro

Prerequisites

Understanding of ASTFamiliarity with ESLint rule structureBasic JavaScript

Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.

75