lint(): fix linter issues, set callable types

nestjs · Mar 15, 2019 · 1476804 · 1476804
1 parent 98cea88
commit 1476804
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/packages/common/utils/merge-with-values.util.ts b/packages/common/utils/merge-with-values.util.ts
@@ -1,6 +1,4 @@
-export interface Constructor<T> {
-  new (...args: any[]): T;
-}
+export type Constructor<T> = new (...args: any[]) => T;
 
 export const MergeWithValues = <T extends Constructor<{}>>(data: {
   [param: string]: any;

diff --git a/packages/core/nest-application-context.ts b/packages/core/nest-application-context.ts
@@ -106,9 +106,7 @@ export class NestApplicationContext implements INestApplicationContext {
           .trim(),
       )
       // filter out the signals which is already listening to
-      .filter(
-        (signal: string) => !this.activeShutdownSignals.includes(signal),
-      ) as string[];
+      .filter(signal => !this.activeShutdownSignals.includes(signal));
 
     this.listenToShutdownSignals(signals);
     return this;

diff --git a/tslint.json b/tslint.json
@@ -14,6 +14,7 @@
     "no-implicit-dependencies": false,
     "no-unnecessary-initializer": false,
     "unified-signatures": false,
+    "callable-types": false,
     "max-line-length": [150],
     "member-ordering": [false],
     "curly": false,