Microsoft/TypeScript

MIDIMessageEvent data is incorrectly typed

Open

#60.546 aperta il 20 nov 2024

Vedi su GitHub
 (1 commento) (1 reazione) (0 assegnatari)TypeScript (6726 fork)batch import
BugDomain: lib.d.tsHelp Wanted

Metriche repository

Star
 (48.455 star)
Metriche merge PR
 (Merge medio 6g 17h) (9 PR mergiate in 30 g)

Descrizione

🔎 Search Terms

In https://github.com/microsoft/TypeScript/pull/57027 MIDIMessageEvent.data was changed from Uint8Array to Uint8Array | null (see https://github.com/microsoft/TypeScript/pull/57027/files#diff-dc0eab937d15e62545da3ed7b4f40ad6b24f15dd88fbc6ceda2bfb4ed8356eb0L14670).

According to https://developer.mozilla.org/en-US/docs/Web/API/MIDIMessageEvent/data data the value will always be an Uint8Array.

🕗 Version & Regression Information

This changed in #57027

⏯ Playground Link

No response

💻 Code

🙁 Actual behavior

'event.data' is possibly 'null'.ts(18047)

🙂 Expected behavior

No warning about data property possibly being null.

Additional information about the issue

No response

Guida contributor

Direzione di ricerca
Individua la definizione del tipo per MIDIMessageEvent.data nei file lib di TypeScript (probabilmente lib.dom.d.ts). Il problema fa riferimento a una modifica introdotta nella PR #57027 che ha aggiunto un'unione con null. Secondo MDN, il valore è sempre Uint8Array, quindi ripristina il tipo a Uint8Array. Verifica se ci sono altri utilizzi che dipendono dall'unione con null. La correzione è una modifica di una riga nel file .d.ts appropriato.
Tech stack
typescript
Dominio
api
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80