build: add prettier and tweak build (#30)

* build: add prettier and fix tslint

* build: add lint/format scipts with prebuild hook

* fix: resolve all tslint issues

* style: apply prettier on whole project

* fix: add missing tslint disable pragma

* build(tools): get rid of ts-node and use vanilla js with proper TS checking

* chore: commit package lock

* ci: execute build script on ci

* style: apply prettier

Author: Hotell

.prettierignore

@@ -0,0 +1,2 @@
+dist
+dist-test

.prettierrc

@@ -0,0 +1,8 @@
+{
+  "semi": true,
+  "printWidth": 140,
+  "trailingComma": "es5",
+  "singleQuote": true,
+  "useTabs": false,
+  "bracketSpacing": true
+}

.travis.yml

@@ -11,3 +11,5 @@ os:
 before_script:
   - npm install
 
+script:
+ - npm run build

README.md

@@ -26,6 +26,7 @@ $ yarn add injection-js
 >
 > For ES5 `Class` syntax and TypeScript you need a polyfill for the [Reflect API](http://www.ecma-international.org/ecma-262/6.0/#sec-reflection).
 > You can use:
+>
 > - [reflect-metadata](https://www.npmjs.com/package/reflect-metadata)
 > - [`core-js` (`core-js/es7/reflect`)](https://www.npmjs.com/package/core-js)
 >
@@ -53,16 +54,11 @@ class Service2 {
   }
 
   createChildInjector(): void {
-    const childInjector = ReflectiveInjector.resolveAndCreate([
-      Service
-    ], this.injector);
+    const childInjector = ReflectiveInjector.resolveAndCreate([Service], this.injector);
   }
 }
 
-const injector = ReflectiveInjector.resolveAndCreate([
-  Service,
-  Http
-]);
+const injector = ReflectiveInjector.resolveAndCreate([Service, Http]);
 
 console.log(injector.get(Service) instanceof Service);
 ```
@@ -96,13 +92,16 @@ require('reflect-metadata');
 var di = require('injection-js');
 
 var Http = di.Class({
-  constructor: function () {}
+  constructor: function() {},
 });
 
 var Service = di.Class({
-  constructor: [Http, function (http) {
-    this.http = http;
-  }]
+  constructor: [
+    Http,
+    function(http) {
+      this.http = http;
+    },
+  ],
 });
 
 var injector = di.ReflectiveInjector.resolveAndCreate([Http, Service]);
@@ -119,4 +118,3 @@ For full documentation check Angular DI docs:
 # License
 
 MIT
-

demo/index-es6.js

@@ -15,4 +15,3 @@ class Service {
 const injector = ReflectiveInjector.resolveAndCreate([Http, Service]);
 
 console.log(injector.get(Service) instanceof Service);
-

demo/index.js

@@ -2,16 +2,18 @@ require('reflect-metadata');
 var di = require('../dist/injection.bundle');
 
 var Http = di.Class({
-  constructor: function () {}
+  constructor: function() {},
 });
 
 var Service = di.Class({
-  constructor: [Http, function (http) {
-    this.http = http;
-  }]
+  constructor: [
+    Http,
+    function(http) {
+      this.http = http;
+    },
+  ],
 });
 
 var injector = di.ReflectiveInjector.resolveAndCreate([Http, Service]);
 
 console.log(injector.get(Service) instanceof Service);
-

lib/facade/errors.ts

@@ -13,7 +13,6 @@ export const ERROR_DEBUG_CONTEXT = 'ngDebugContext';
 export const ERROR_ORIGINAL_ERROR = 'ngOriginalError';
 export const ERROR_LOGGER = 'ngErrorLogger';
 
-
 export function getType(error: Error): Function {
   return (error as any)[ERROR_TYPE];
 }
@@ -35,8 +34,7 @@ export function getErrorLogger(error: Error): (console: Console, ...values: any[
 }
 
 export function wrappedError(message: string, originalError: any): Error {
-  const msg =
-      `${message} caused by: ${originalError instanceof Error ? originalError.message: originalError }`;
+  const msg = `${message} caused by: ${originalError instanceof Error ? originalError.message : originalError}`;
   const error = Error(msg);
   (error as any)[ERROR_ORIGINAL_ERROR] = originalError;
   return error;

lib/facade/lang.ts

@@ -15,7 +15,7 @@ export interface BrowserNodeGlobal {
   Date: DateConstructor;
   RegExp: RegExpConstructor;
   JSON: typeof JSON;
-  Math: any;  // typeof Math;
+  Math: any; // typeof Math;
   assert(condition: any): void;
   Reflect: any;
   getAngularTestability: Function;
@@ -52,7 +52,7 @@ if (typeof window === 'undefined') {
 // exports the original value of the symbol.
 const _global: BrowserNodeGlobal = globalScope;
 
-export {_global as global};
+export { _global as global };
 
 export function isPresent(obj: any): boolean {
   return obj != null;
@@ -83,11 +83,11 @@ export function stringify(token: any): string {
 export abstract class DebugContext {
   // We don't really need this
   // abstract get view(): ViewData;
-  abstract get nodeIndex(): number|null;
+  abstract get nodeIndex(): number | null;
   abstract get injector(): Injector;
   abstract get component(): any;
   abstract get providerTokens(): any[];
-  abstract get references(): {[key: string]: any};
+  abstract get references(): { [key: string]: any };
   abstract get context(): any;
   abstract get componentRenderElement(): any;
   abstract get renderNode(): any;

lib/facade/type.ts

@@ -22,4 +22,6 @@ export function isType(v: any): v is Type<any> {
   return typeof v === 'function';
 }
 
-export interface Type<T> extends Function { new (...args: any[]): T; }
+export interface Type<T> extends Function {
+  new (...args: any[]): T;
+}

lib/forward_ref.ts

@@ -6,9 +6,8 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {stringify} from './facade/lang';
-import {Type} from './facade/type';
-
+import { stringify } from './facade/lang';
+import { Type } from './facade/type';
 
 /**
  * An interface that a function passed into {@link forwardRef} has to implement.
@@ -18,7 +17,9 @@ import {Type} from './facade/type';
  * {@example core/di/ts/forward_ref/forward_ref_spec.ts region='forward_ref_fn'}
  * @experimental
  */
-export interface ForwardRefFn { (): any; }
+export interface ForwardRefFn {
+  (): any;
+}
 
 /**
  * Allows to refer to references which are not yet defined.
@@ -34,8 +35,10 @@ export interface ForwardRefFn { (): any; }
  */
 export function forwardRef(forwardRefFn: ForwardRefFn): Type<any> {
   (<any>forwardRefFn).__forward_ref__ = forwardRef;
-  (<any>forwardRefFn).toString = function() { return stringify(this()); };
-  return (<Type<any>><any>forwardRefFn);
+  (<any>forwardRefFn).toString = function() {
+    return stringify(this());
+  };
+  return <Type<any>>(<any>forwardRefFn);
 }
 
 /**
@@ -51,8 +54,7 @@ export function forwardRef(forwardRefFn: ForwardRefFn): Type<any> {
  * @experimental
  */
 export function resolveForwardRef(type: any): any {
-  if (typeof type === 'function' && type.hasOwnProperty('__forward_ref__') &&
-      type.__forward_ref__ === forwardRef) {
+  if (typeof type === 'function' && type.hasOwnProperty('__forward_ref__') && type.__forward_ref__ === forwardRef) {
     return (<ForwardRefFn>type)();
   } else {
     return type;

lib/index.ts

@@ -14,13 +14,13 @@
 
 export * from './metadata';
 
-export {forwardRef, resolveForwardRef, ForwardRefFn} from './forward_ref';
+export { forwardRef, resolveForwardRef, ForwardRefFn } from './forward_ref';
 
-export {Injector} from './injector';
-export {ReflectiveInjector} from './reflective_injector';
-export {Provider, TypeProvider, ValueProvider, ClassProvider, ExistingProvider, FactoryProvider} from './provider';
-export {ResolvedReflectiveFactory, ResolvedReflectiveProvider} from './reflective_provider';
-export {ReflectiveKey} from './reflective_key';
-export {InjectionToken, OpaqueToken} from './injection_token';
-export {Class, TypeDecorator, makeDecorator} from './util/decorators';
-export {Type} from './facade/type';
+export { Injector } from './injector';
+export { ReflectiveInjector } from './reflective_injector';
+export { Provider, TypeProvider, ValueProvider, ClassProvider, ExistingProvider, FactoryProvider } from './provider';
+export { ResolvedReflectiveFactory, ResolvedReflectiveProvider } from './reflective_provider';
+export { ReflectiveKey } from './reflective_key';
+export { InjectionToken, OpaqueToken } from './injection_token';
+export { Class, TypeDecorator, makeDecorator } from './util/decorators';
+export { Type } from './facade/type';

lib/injection_token.ts

@@ -32,7 +32,9 @@
 export class OpaqueToken {
   constructor(protected _desc: string) {}
 
-  toString(): string { return `Token ${this._desc}`; }
+  toString(): string {
+    return `Token ${this._desc}`;
+  }
 }
 
 /**
@@ -61,7 +63,11 @@ export class InjectionToken<T> extends OpaqueToken {
   // This unused property is needed here so that TS can differentiate InjectionToken from
   // OpaqueToken since otherwise they would have the same shape and be treated as equivalent.
   private _differentiate_from_OpaqueToken_structurally: any;
-  constructor(desc: string) { super(desc); }
+  constructor(desc: string) {
+    super(desc);
+  }
 
-  toString(): string { return `InjectionToken ${this._desc}`; }
+  toString(): string {
+    return `InjectionToken ${this._desc}`;
+  }
 }

lib/injector.ts

@@ -6,14 +6,15 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {stringify} from './facade/lang';
-import {Type} from './facade/type';
+import { stringify } from './facade/lang';
+import { Type } from './facade/type';
 
-import {InjectionToken} from './injection_token';
+import { InjectionToken } from './injection_token';
 
 const _THROW_IF_NOT_FOUND = new Object();
 export const THROW_IF_NOT_FOUND = _THROW_IF_NOT_FOUND;
 
+// tslint:disable-next-line:class-name no-use-before-declare
 class _NullInjector implements Injector {
   get(token: any, notFoundValue: any = _THROW_IF_NOT_FOUND): any {
     if (notFoundValue === _THROW_IF_NOT_FOUND) {
@@ -54,7 +55,7 @@ export abstract class Injector {
    * Injector.THROW_IF_NOT_FOUND is given
    * - Returns the `notFoundValue` otherwise
    */
-  abstract get<T>(token: Type<T>|InjectionToken<T>, notFoundValue?: T): T;
+  abstract get<T>(token: Type<T> | InjectionToken<T>, notFoundValue?: T): T;
   /**
    * @deprecated from v4.0.0 use Type<T> or InjectionToken<T>
    * @suppress {duplicate}

lib/metadata.ts

@@ -6,8 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {makeDecorator, makeParamDecorator} from './util/decorators';
-
+import { makeDecorator, makeParamDecorator } from './util/decorators';
 
 /**
  * Type of the Inject decorator / constructor function.
@@ -50,7 +49,9 @@ export interface InjectDecorator {
  *
  * @stable
  */
-export interface Inject { token: any; }
+export interface Inject {
+  token: any;
+}
 
 /**
  * Inject decorator and metadata.
@@ -60,7 +61,6 @@ export interface Inject { token: any; }
  */
 export const Inject: InjectDecorator = makeParamDecorator('Inject', [['token', undefined]]);
 
-
 /**
  * Type of the Optional decorator / constructor function.
  *
@@ -197,7 +197,6 @@ export interface Self {}
  */
 export const Self: SelfDecorator = makeParamDecorator('Self', []);
 
-
 /**
  * Type of the SkipSelf decorator / constructor function.
  *

lib/provider.ts

@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {Type} from './facade/type';
+import { Type } from './facade/type';
 
 /**
  * @whatItDoes Configures the {@link Injector} to return an instance of `Type` when `Type' is used
@@ -216,5 +216,4 @@ export interface FactoryProvider {
  *
  * @stable
  */
-export type Provider =
-    TypeProvider | ValueProvider | ClassProvider | ExistingProvider | FactoryProvider | any[];
+export type Provider = TypeProvider | ValueProvider | ClassProvider | ExistingProvider | FactoryProvider | any[];

lib/reflection/platform_reflection_capabilities.ts

@@ -6,21 +6,21 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {Type} from '../facade/type';
-import {GetterFn, MethodFn, SetterFn} from './types';
+import { Type } from '../facade/type';
+import { GetterFn, MethodFn, SetterFn } from './types';
 
 export interface PlatformReflectionCapabilities {
   isReflectionEnabled(): boolean;
   factory(type: Type<any>): Function;
   hasLifecycleHook(type: any, lcProperty: string): boolean;
   parameters(type: Type<any>): any[][];
   annotations(type: Type<any>): any[];
-  propMetadata(typeOrFunc: Type<any>): {[key: string]: any[]};
+  propMetadata(typeOrFunc: Type<any>): { [key: string]: any[] };
   getter(name: string): GetterFn;
   setter(name: string): SetterFn;
   method(name: string): MethodFn;
   importUri(type: Type<any>): string;
   resourceUri(type: Type<any>): string;
-  resolveIdentifier(name: string, moduleUrl: string, members: string[]|null, runtime: any): any;
+  resolveIdentifier(name: string, moduleUrl: string, members: string[] | null, runtime: any): any;
   resolveEnum(enumIdentifier: any, name: string): any;
 }

lib/reflection/reflection.ts

@@ -6,10 +6,10 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {ReflectionCapabilities} from './reflection_capabilities';
-import {Reflector} from './reflector';
+import { ReflectionCapabilities } from './reflection_capabilities';
+import { Reflector } from './reflector';
 
-export {Reflector} from './reflector';
+export { Reflector } from './reflector';
 
 /**
  * The {@link Reflector} used internally in Angular to access metadata