From 0b9cb255b954886b0f100e966c2ddb841c91015e Mon Sep 17 00:00:00 2001 From: Tabeth Nkangoh Date: Fri, 16 Feb 2018 10:29:38 -0500 Subject: [PATCH] Update index.d.ts There appears to be a discrepancy between the description in the comments as well as [the documentation](https://www.emberjs.com/api/ember/3.0/classes/MutableArray/methods/removeAt?anchor=removeAt) and the typing itself. Since we can pass either a single index **or** a start and length, `len` should be an optional parameter (to handle the scenario where you just pass a single index). --- types/ember/index.d.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/types/ember/index.d.ts b/types/ember/index.d.ts index 4a3d7d45d1..9f08d2fc24 100755 --- a/types/ember/index.d.ts +++ b/types/ember/index.d.ts @@ -1523,7 +1523,7 @@ declare module 'ember' { * Remove an object at the specified index using the `replace()` primitive * method. You can pass either a single index, or a start and a length. */ - removeAt(start: number, len: number): this; + removeAt(start: number, len?: number): this; /** * Push the object onto the end of the array. Works just like `push()` but it * is KVO-compliant.