josdejong/mathjs

Insufficient typing for `add`, `subtract`, `dotMultiply` etc

Open

#2349 aperta il 9 nov 2021

Vedi su GitHub
 (0 commenti) (3 reazioni) (0 assegnatari)JavaScript (1298 fork)batch import
help wantedtypescript

Metriche repository

Star
 (13.832 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

The functions

lack sufficiently fine-grained type definitions. In particular, this produces an error:

math.add([1, 2], [3, 4])[0];
// Element implicitly has an 'any' type because expression
// of type '0' can't be used to index type 'MathType'.
// Property '0' does not exist on type 'MathType'

while it should be clear that the result is a one-dimensional array.

Adding some overloads would make the TypeScript experience with Mathjs much smoother.

Guida contributor

Direzione di ricerca
Esamina le definizioni di tipo in `types/index.d.ts` per le funzioni `add`, `subtract`, `dotDivide`, `dotMultiply` e `dotPow`. Comprendi l'attuale tipo unione `MathType` e come gli overload possono specificare tipi di ritorno più precisi. Guarda gli overload esistenti nello stesso file per orientarti. Aggiungi overload che deducano correttamente il tipo del risultato quando gli input sono array.
Tech stack
javascripttypescript
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptJavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75