Linting

Author: Xapphire13

package-lock.json

@@ -5,8 +5,9 @@
   "main": "dist/index.js",
   "scripts": {
     "build": "rimraf ./dist ./types && tsc -p ./src",
-    "test": "jest",
+    "test": "npm run lint && jest",
     "test:coverage": "jest --coverage",
+    "lint": "tslint -p src && tslint -p tests",
     "prepush": "npm test"
   },
   "repository": {
@@ -67,6 +68,8 @@
     "jest": "^22.2.2",
     "rimraf": "^2.6.2",
     "ts-jest": "^22.0.4",
+    "tslint": "^5.9.1",
+    "tslint-eslint-rules": "^5.0.0",
     "typescript": "^2.6.1"
   }
 }

package.json

@@ -1,9 +1,9 @@
 import "reflect-metadata";
 
-import {instance as globalContainer, typeInfo} from "./dependency-container";
-import { InjectionToken, Provider } from "./providers";
-import { constructor, RegistrationOptions } from "./types";
 import {INJECTION_TOKEN_METADATA_KEY, getParamInfo} from "./reflection-helpers";
+import {InjectionToken, Provider} from "./providers";
+import {RegistrationOptions, constructor} from "./types";
+import {instance as globalContainer, typeInfo} from "./dependency-container";
 
 /**
  * Class decorator factory that allows the class' dependencies to be injected
@@ -12,9 +12,9 @@ import {INJECTION_TOKEN_METADATA_KEY, getParamInfo} from "./reflection-helpers";
  * @return {Function} The class decorator
  */
 export function injectable<T>(): (target: constructor<T>) => void {
-    return function(target: constructor<T>): void {
-        typeInfo.set(target, getParamInfo(target));
-    };
+  return function(target: constructor<T>): void {
+    typeInfo.set(target, getParamInfo(target));
+  };
 }
 
 /**
@@ -26,15 +26,15 @@ export function injectable<T>(): (target: constructor<T>) => void {
  * @return {Function} The class decorator
  */
 export function autoInjectable(): (target: constructor<any>) => any {
-    return function(target: constructor<any>): constructor<any> {
-        const paramInfo = getParamInfo(target);
+  return function(target: constructor<any>): constructor<any> {
+    const paramInfo = getParamInfo(target);
 
-        return class extends target {
-          constructor(...args: any[]) {
-              super(...args.concat(paramInfo.slice(args.length).map(type => globalContainer.resolve(type))));
-          }
-        }
-    }
+    return class extends target {
+      constructor(...args: any[]) {
+        super(...args.concat(paramInfo.slice(args.length).map(type => globalContainer.resolve(type))));
+      }
+    };
+  };
 }
 
 /**
@@ -43,22 +43,22 @@ export function autoInjectable(): (target: constructor<any>) => any {
  * @return {Function} The parameter decorator
  */
 export function inject(token: InjectionToken<any>): (target: any, propertyKey: string | symbol, parameterIndex: number) => any {
-    return function(target: any, _propertyKey: string | symbol, parameterIndex: number): any {
-        const injectionTokens = Reflect.getOwnMetadata(INJECTION_TOKEN_METADATA_KEY, target) || {};
-        injectionTokens[parameterIndex] = token;
-        Reflect.defineMetadata(INJECTION_TOKEN_METADATA_KEY, injectionTokens, target);
-    };
+  return function(target: any, _propertyKey: string | symbol, parameterIndex: number): any {
+    const injectionTokens = Reflect.getOwnMetadata(INJECTION_TOKEN_METADATA_KEY, target) || {};
+    injectionTokens[parameterIndex] = token;
+    Reflect.defineMetadata(INJECTION_TOKEN_METADATA_KEY, injectionTokens, target);
+  };
 }
 
 /**
  * Class decorator factory that allows constructor dependencies to be registered at runtime.
  *
  * @return {Function} The class decorator
  */
-export function registry(registrations: ({token: InjectionToken, options?: RegistrationOptions} & Provider<any>)[] = []): (target: any) => any {
-    return function(target: any): any {
-        registrations.forEach(({token, options, ...provider}) => globalContainer.register(token, <any>provider, options));
+export function registry(registrations: ({ token: InjectionToken, options?: RegistrationOptions } & Provider<any>)[] = []): (target: any) => any {
+  return function(target: any): any {
+    registrations.forEach(({token, options, ...provider}) => globalContainer.register(token, <any>provider, options));
 
-        return target;
-    };
+    return target;
+  };
 }