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

Metriche repository

Star

5022 stelle (5022 stelle)

Metriche merge PR

Merge medio 4h 30m (Merge medio 4h 30m)26 PR mergiate in 30 g (26 PR mergiate in 30 g)

Descrizione

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

Promise.resolve() // Promise<void>

Guida contributor

Direzione di ricerca

Analizza la regola `no useless undefined` in `rules/no useless undefined.js`. Attualmente la regola rimuove gli argomenti `undefined` dalle chiamate di funzione, ma per `Promise.resolve(undefined)`, rimuovere `undefined` cambia il tipo di ritorno da `Promise<undefined>` a `Promise<void>`. La regola dovrebbe essere aggiornata per preservare `undefined` quando influisce sulla semantica dei tipi, possibilmente controllando se la firma della funzione si aspetta `undefined` o se il contesto coinvolge `Promise`. Rivedi la logica della regola per la rimozione di `undefined` e aggiungi un'eccezione per `Promise.resolve` e casi simili in cui `undefined` è significativo.

Tech stack

javascript

Dominio

developer experiencetooling

Tipo issue

Bug

DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.

3

Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.

1-3 ore

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

ESLint plugin basicsJavaScript Promise types

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

60