Preserve `set`-only accessors in declaration emit

Aperta
#58,119 0 commenti 6 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
48/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
typescript
Ambito
compilers

Direzione di ricerca

Inizia dal Playground collegato e riproduci l'output della dichiarazione per gli accessor same, divergent, readonly e writeonly. Segui il percorso di emissione delle dichiarazioni di TypeScript per gli accessor degli oggetti e confronta i test o le baseline esistenti relativi agli accessor; il lavoro è completo quando un accessor write-only viene preservato nella dichiarazione emessa senza modificare gli altri output mostrati.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

In Discussion Suggestion

Playground Link

Related to https://github.com/microsoft/TypeScript/issues/58112#issuecomment-2043419380

export let same = {
    set x(value: string) {
    },
    get x(): string {
        throw new Error("Not implemented.")
    },
};

export let divergent = {
    set x(value: string) {
    },
    get x(): number {
        throw new Error("Not implemented.")
    },
};

export let readonly = {
    get x(): string {
        throw new Error("Not implemented.")
    },
};

export let writeonly = {
    set x(value: string) {
    },
};

Current

Notice that in the declaration of writeonly, we don't preserve the accessor at all.

export declare let same: {
    x: string;
};
export declare let divergent: {
    get x(): number;
    set x(value: string);
};
export declare let readonly: {
    readonly x: string;
};
export declare let writeonly: {
    x: string;
};

Expected

If we had a writeonly modifier, I'd argue we could just use that as a parallel to the readonly declaration; however, we do support accessors in declaration emit, and we could emit that in the case of a write-only accessor.

It is technically a breaking change though for consumers of older TypeScript versions, and the only way to opt out would be to declare a no-op get accessor.

export declare let same: {
    x: string;
};
export declare let divergent: {
    get x(): number;
    set x(value: string);
};
export declare let readonly: {
    readonly x: string;
};
export declare let writeonly: {
    set x(): string;
};
Lingua principale
Go
Stelle
111k
Fork
14.4k
Merge medio
1g 19h
PR unite (30g)
117

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di microsoft/TypeScript

Tutte le issue di microsoft/TypeScript

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.