Microsoft/TypeScript

MIDIMessageEvent data is incorrectly typed

Open

#60 546 ouverte le 20 nov. 2024

Voir sur GitHub
 (1 commentaire) (1 réaction) (0 assignés)TypeScript (6 726 forks)batch import
BugDomain: lib.d.tsHelp Wanted

Métriques du dépôt

Stars
 (48 455 stars)
Métriques de merge PR
 (Merge moyen 6j 17h) (9 PRs mergées en 30 j)

Description

🔎 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

Guide contributeur

Direction de recherche
Localisez la définition de type pour MIDIMessageEvent.data dans les fichiers lib de TypeScript (probablement lib.dom.d.ts). Le problème pointe vers un changement introduit dans la PR #57027 qui a ajouté une union avec null. Selon MDN, la valeur est toujours Uint8Array, donc rétablissez le type à Uint8Array. Vérifiez s'il existe d'autres utilisations qui dépendent de l'union avec null. La correction est un changement d'une ligne dans le fichier .d.ts approprié.
Stack technique
typescript
Domaine
api
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80