microsoft / microsoft/TypeScript

Type check TS file instead of index.d.ts in node_modules leads to build error that cannot be ignored

Offen
#41,487 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Needs Investigation
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.3k
Ø Merge
1 T. 19 Std.
Gemergte PRs (30 T.)
117

Beschreibung

TypeScript Version: 4.1 RC

Search Terms: node_module check

Code

Run tsc on this PR:

https://github.com/DimensionDev/Maskbook/pull/1879

Emitting this error (I have skipLibCheck on but tsc is going to read .tssource file not.d.ts` file):

node_modules/promievent/PromiEvent.ts:28:33 - error TS2345: Argument of type '(resolve: Resolve<T>, reject: Reject) => void' is not assignable to parameter of type '(resolve: (value: T | PromiseLike<T>) => void, reject: (reason?: any) => void) => void'.
  Types of parameters 'resolve' and 'resolve' are incompatible.
    Types of parameters 'value' and 'value' are incompatible.
      Type 'T | PromiseLike<T> | undefined' is not assignable to type 'T | PromiseLike<T>'.
        Type 'undefined' is not assignable to type 'T | PromiseLike<T>'.

28      this.promise = new Promise<T>(executor)
                                      ~~~~~~~~


Found 1 error.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Reproduziere den Bericht, indem du tsc auf DimensionDev/Maskbook PR 1879 mit TypeScript 4.1 RC und aktiviertem skipLibCheck ausführst. Beginne mit der Diagnose in node_modules/promievent/PromiEvent.ts und vergleiche, warum die .ts-Quelle statt der index.d.ts des Pakets geprüft wird. Als abgeschlossen gilt die Aufgabe, wenn der gemeldete Build-Fehler unter der angegebenen Konfiguration nicht mehr erzeugt wird und, falls ein bestehender Speicherort für Compiler-Tests identifiziert wird, eine Regressionstestabdeckung hinzugefügt wurde.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
compilers
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.