microsoft / microsoft/TypeScript

Weak callback parameters length check in js

Offen
#42,107 2 Kommentare 2 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@sandersn arbeitet bereits daran.

Seit 28.12.2020.

Experimentation Needed Suggestion
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.4k
Ø Merge
1 T. 19 Std.
Gemergte PRs (30 T.)
117

Beschreibung

TypeScript Version: 4.2.0-dev.20201222

Search Terms: callback parameters length

Code

in .ts everything is ok:

type Test = (a: number, b: number) => number

const a = (func: Test) => func

// OK
const b = a((d, e) => d + e)

// OK: Argument of type '(d: any, e: any, f: any) => any' is not assignable to parameter of type 'Test'
const c = a((d, e, f) => d + e + f)

in .js there's no error if parameters length is wrong:

/** @typedef {(a: number, b: number) => number} Test */

/** @type {(func: Test) => Test} */
const a = func => func

// OK
const b = a((d, e) => d + e)

// Actually: Parameter 'd' implicitly has an 'any' type.
// Expected: Argument of type '(d: any, e: any, f: any) => any' is not assignable to parameter of type 'Test'.
const c = a((d, e, f) => d + e + f)

Expected behavior:
Type checker to show the same error in .js file as in .ts file

Actual behavior:
No errors in .js file

Related Issues:
https://github.com/microsoft/TypeScript/issues/40205

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.