microsoft / microsoft/TypeScript

Dynamic import breaks global onError with requirejs

Offen
#32,103 0 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@rbuckton arbeitet bereits daran.

Seit 26.6.2019.

Needs Investigation
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.3k
Ø Merge
2 T. 4 Std.
Gemergte PRs (30 T.)
132

Beschreibung

TypeScript Version: 3.2.2

Search Terms: dynamic import, requirejs, amd, onerror

Code

import('module').then(()=>{ /* some code*/ });

compiled to

new Promise(function (resolve_1, reject_1) {
   require(['module'], resolve_1, reject_1);
});

If require are called with 3 arguments then require.onError is not fired. Reject should be adeed only in case when we have a catch call after import.

Some propose:

//@ts-import-no-reject
import('module').then(()=>{ /* some code*/ });
new Promise(function (resolve_1) {
   require(['module'], resolve_1);
});

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.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

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