microsoft / microsoft/TypeScript

Proposal: allow ambient property declarations to override accessors as well

Aperta
#40,220 3 commenti 19 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

In Discussion Suggestion
Lingua principale
Go
Stelle
111k
Fork
14.4k
Merge medio
1g 19h
PR unite (30g)
117

Descrizione

Currently, this pattern gives an error, but arguably shouldn't:

class B {
  get p() { return B.s } // inferred p: {}
  static get s() { return {} }
}
class C extends B {
  declare readonly p: { x: 1 }
}

Expected behavior:

No error, and C.p: { x: 1 } (but is still an accessor for emit and checking purposes). That is, this should be legal:

class D extends C {
  get p() { return { x: 1 } as const }
}

Actual behavior:

Error, and C.p is a property, not an accessor.

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.

Direzione di ricerca

Inizia riproducendo gli esempi di classi nell’issue e confronta la diagnostica attuale con il comportamento previsto. La modifica è completa quando la dichiarazione ambient può sovrascrivere l’accessor ereditato senza errori, C.p ha il tipo dichiarato e D può ancora dichiarare l’accessor mostrato.

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

Valutazione

Stack tecnologico
typescript
Ambito
compilers
Tipo di issue
Funzionalità
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.