microsoft / microsoft/TypeScript
Vanilla JS: Mixins break inherited type hints
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.4k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
Does this issue occur when all extensions are disabled?: Yes
- VS Code Version: 1.84.2
- OS Version: Windows 10
The issue is quite simple, as shown below, mixins break inherited type hints. exampleMethod() below should have hint references for this.myStr and this.propIsHinted but does not.
Apologies if this belongs somewhere else like the typescript repo, it's vanilla JS but it's probably related to typescript/JSDoc functionality.
function NumberMixin( Base ) {
return class NumberMixin extends Base {
/** @type {number} */
myNum;
};
}
function StringMixin( Base ) {
return class StringMixin extends Base {
/** @type {string} */
myStr;
};
}
class MyBase {
/** @type {boolean} */
propIsHinted;
}
class SuperClass extends NumberMixin(StringMixin(MyBase)) {
exampleMethod() {
// Hinted
this.myNum;
// Not hinted
this.myStr;
// Not hinted
this.propIsHinted;
}
}
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Esegui l’esempio di mixin JavaScript fornito nel contesto TypeScript o VS Code per riprodurre i suggerimenti mancanti. Inizia tracciando l’inferenza dei tipi JSDoc per le classi ereditate e i mixin annidati; il lavoro è completato quando exampleMethod() fornisce suggerimenti per myNum, myStr e propIsHinted.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript, typescript
- Ambito
- compilers
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 38/100