microsoft / microsoft/TypeScript
Not currectly parse superClass starts with `{}`
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 2g 4h
- PR unite (30g)
- 132
Descrizione
### 🔎 Search Terms
"superClass"
### 🕗 Version & Regression Information
- This changed between versions ______ and _______
- This changed in commit or PR _______
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
- I was unable to test this on prior versions because _______
### ⏯ Playground Link
_No response_
### 💻 Code
```ts
class A extends {}.Foo {}
```
```ts
class A extends {}! {}
```
### 🙁 Actual behavior
They are not correctly parsed as super class.
### 🙂 Expected behavior
They should both parse as superClass.
### Additional information about the issue
The following are correctly parsed.
```
class A extends {} {}
```
```
class A extends {Foo}.Foo {}
```
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
Riproduci le due dichiarazioni di classe segnalate e confronta il loro parsing con gli esempi funzionanti nell'issue. Traccia la gestione da parte del parser TypeScript delle clausole di ereditarietà delle classi e delle espressioni calcolate o con asserzione di non nullità, quindi aggiungi una copertura di regressione per entrambe le forme. Il lavoro è completato quando entrambi gli snippet vengono analizzati con la superclasse prevista e gli esempi esistenti rimangono invariati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- compilers
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 42/100