microsoft / microsoft/TypeScript
Change default inference from arrays to tuples and primitives to literals
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
I am working on functional project now and I'm getting a lot of errors related to too wide type inference for values. It looks like
function test(a: { data: 1 | 2 }) { return a }
const a = { data: 2 }
// Types of property 'data' are incompatible.
// Type 'number' is not assignable to type '2'.ts(2345)
const b = test(a)
There're really many errors like this, and the main problem is that you can convert type A to type B, but not vice versa.
type A = { data: 2 }
type B = { data: number }
So, I ask you to add a new compiler option (to not break existing code) that will make type checker infer types as narrow as possible. It will be very helpful for functional programming.
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
Inizia esaminando la documentazione del type checker di TypeScript e delle opzioni del compilatore, usando come comportamento previsto gli esempi dell'issue relativi ad array, primitivi e inferenza dei literal. Definisci prima l'ambito dell'opzione e le regole di compatibilità; il lavoro è completo quando l'inferenza ristretta richiesta funziona senza modificare il comportamento esistente quando l'opzione è disabilitata.
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
- 30/100