chore: turn on noImplicitThis (#1842)

faker-js · Feb 20, 2023 · 0eba320 · 0eba320
1 parent be5bd5c
commit 0eba320
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/src/modules/helpers/index.ts b/src/modules/helpers/index.ts
@@ -915,7 +915,7 @@ export class HelpersModule {
     ) => RecordKey
   >(
     method: Method,
-    args?: Parameters<Method>,
+    args: Parameters<Method> = [] as Parameters<Method>,
     options: {
       /**
        * This parameter does nothing.

diff --git a/src/modules/helpers/unique.ts b/src/modules/helpers/unique.ts
@@ -143,7 +143,7 @@ export function exec<
   }
 
   // Execute the provided method to find a potential satisfied value.
-  const result: ReturnType<Method> = method.apply(this, args);
+  const result: ReturnType<Method> = method(...args) as ReturnType<Method>;
 
   // If the result has not been previously found, add it to the found array and return the value as it's unique.
   if (compare(store, result) === -1 && exclude.indexOf(result) === -1) {

diff --git a/tsconfig.json b/tsconfig.json
@@ -10,6 +10,7 @@
     "alwaysStrict": true,
     "strictFunctionTypes": true,
     "noImplicitAny": true,
+    "noImplicitThis": true,
     "useUnknownInCatchVariables": true,
     "stripInternal": true,
     "baseUrl": "."