[no-useless-undefined] incorrect fix for ts `Promise.resolve(undefined)`

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

Promise.resolve(undefined) // Promise<undefined>

Promise.resolve() // Promise<void>

Guia do colaborador

Direção de pesquisa

Investigue a regra `no useless undefined` em `rules/no useless undefined.js`. Atualmente, a regra remove argumentos `undefined` de chamadas de função, mas para `Promise.resolve(undefined)`, remover `undefined` altera o tipo de retorno de `Promise<undefined>` para `Promise<void>`. A regra deve ser atualizada para preservar `undefined` quando isso afeta a semântica de tipos, possivelmente verificando se a assinatura da função espera `undefined` ou se o contexto envolve `Promise`. Revise a lógica da regra para remover `undefined` e adicione uma exceção para `Promise.resolve` e casos semelhantes onde `undefined` é significativo.

Pilha de tecnologia

javascript

Domain

developer experiencetooling

Tipo Issue

Bug

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.

Antigo

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

Claro

Prerequisites

ESLint plugin basicsJavaScript Promise types

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

60