sindresorhus/eslint-plugin-unicorn

[consistent-destructuring] incorrect fix for ts `in` type guard

Chiusa

#1112 aperta il 18 feb 2021

Vedi su GitHubGuida contributor
 (1 commento) (1 reazione) (0 assegnatari)JavaScript (468 fork)user submission
bughelp wanted

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 1g 16h) (399 PR mergiate in 30 g)

Descrizione

const { a }: { a: string } & ({ b: number } | { c: number } | {}) = params
const value = 'b' in params ? params.b : 'c' in params ? params.c : null

// incorrect fixed
const { a, b, c }: { a: string } & ({ b: number } | { c: number } | {}) = params
const value = 'b' in params ? b : 'c' in params ? c : null

Reference: https://www.typescriptlang.org/docs/handbook/advanced-types.html#type-guards-and-differentiating-types

The fixed codes is not runnable anymore. Playground

Guida contributor

Direzione di ricerca
Esamina il sorgente della regola consistent destructuring in rules/consistent destructuring.js. Attualmente la regola destruttura erroneamente le proprietà quando si utilizza la type guard 'in'. La correzione dovrebbe rilevare quando il lato destro dell'operatore 'in' è un parametro di un pattern di destrutturazione ed evitare di destrutturare quella proprietà. Il link al playground fornisce una riproduzione chiara. Controlla anche i test della regola per assicurarti che la correzione copra i casi limite.
Tech stack
javascripttypescriptnodejs
Dominio
toolingdeveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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 rule developmentTypeScript type guardsAST
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50