fix: make ng build pass

ng-package.json

@@ -4,6 +4,9 @@
   "deleteDestPath": false,
   "whitelistedNonPeerDependencies": ["moment"],
   "lib": {
-    "entryFile": "src/index.ts"
+    "entryFile": "src/index.ts",
+    "umdModuleIds": {
+      "moment": "moment"
+    }
   }
 }

src/add.pipe.ts

@@ -1,14 +1,16 @@
 /* ngx-moment (c) 2015, 2016 Uri Shaked / MIT Licence */
 
-import {Pipe, ChangeDetectorRef, PipeTransform} from '@angular/core';
+import {Pipe, PipeTransform} from '@angular/core';
 import * as moment from 'moment';
 
+const momentConstructor = moment;
+
 @Pipe({ name: 'amAdd' })
 export class AddPipe implements PipeTransform {
     transform(value: any, amount: moment.DurationInputArg1, unit?: moment.DurationInputArg2): any {
         if (typeof amount === 'undefined' || (typeof amount === 'number' && typeof unit === 'undefined')) {
             throw new Error('AddPipe: missing required arguments');
         }
-        return moment(value).add(amount, unit);
+        return momentConstructor(value).add(amount, unit);
     }
 }

src/calendar.pipe.ts

@@ -4,8 +4,7 @@ import { Pipe, ChangeDetectorRef, PipeTransform, EventEmitter, OnDestroy, NgZone
 import * as moment from 'moment';
 import { Subscription } from 'rxjs';
 
-// under systemjs, moment is actually exported as the default export, so we account for that
-const momentConstructor: (value?: any) => moment.Moment = (<any>moment).default || moment;
+const momentConstructor = moment;
 
 @Pipe({ name: 'amCalendar', pure: false })
 export class CalendarPipe implements PipeTransform, OnDestroy {

src/date-format.pipe.ts

@@ -1,10 +1,9 @@
 /* ngx-moment (c) 2015, 2016 Uri Shaked / MIT Licence */
 
-import {Pipe, ChangeDetectorRef, PipeTransform} from '@angular/core';
+import {Pipe, PipeTransform} from '@angular/core';
 import * as moment from 'moment';
 
-// under systemjs, moment is actually exported as the default export, so we account for that
-const momentConstructor: (value?: any) => moment.Moment = (<any>moment).default || moment;
+const momentConstructor = moment;
 
 @Pipe({ name: 'amDateFormat' })
 export class DateFormatPipe implements PipeTransform {

src/difference.pipe.spec.ts

@@ -1,5 +1,4 @@
 import {DifferencePipe} from './difference.pipe';
-import * as moment from 'moment';
 
 describe('DifferencePipe', () => {
   let pipe: DifferencePipe;

src/difference.pipe.ts

@@ -1,10 +1,9 @@
 /* ngx-moment (c) 2015, 2016 Uri Shaked / MIT Licence */
 
-import {Pipe, ChangeDetectorRef, PipeTransform} from '@angular/core';
+import {Pipe, PipeTransform} from '@angular/core';
 import * as moment from 'moment';
 
-// under systemjs, moment is actually exported as the default export, so we account for that
-const momentConstructor: (value?: any) => moment.Moment = (<any>moment).default || moment;
+const momentConstructor = moment;
 
 @Pipe({ name: 'amDifference' })
 export class DifferencePipe implements PipeTransform {

src/from-unix.pipe.ts

@@ -1,6 +1,6 @@
 /* ngx-moment (c) 2015, 2016 Uri Shaked / MIT Licence */
 
-import {Pipe, ChangeDetectorRef, PipeTransform} from '@angular/core';
+import {Pipe, PipeTransform} from '@angular/core';
 import * as moment from 'moment';
 
 @Pipe({ name: 'amFromUnix' })

src/local.pipe.ts

@@ -1,12 +1,11 @@
 import { Pipe, PipeTransform } from '@angular/core';
 import * as moment from 'moment';
 
-// under systemjs, moment is actually exported as the default export, so we account for that
-const momentConstructor: (value?: any) => moment.Moment = (<any>moment).default || moment;
+const momentConstructor = moment;
 
 @Pipe({ name: 'amLocal' })
 export class LocalTimePipe implements PipeTransform {
     transform(value: Date | moment.Moment | string | number): moment.Moment {
-        return moment(value).local();
+        return momentConstructor(value).local();
     }
 }

src/locale.pipe.ts

@@ -2,11 +2,11 @@ import { Pipe, PipeTransform } from '@angular/core';
 import * as moment from 'moment';
 
 // under systemjs, moment is actually exported as the default export, so we account for that
-const momentConstructor: (value?: any) => moment.Moment = (<any>moment).default || moment;
+const momentConstructor = moment;
 
 @Pipe({ name: 'amLocale' })
 export class LocalePipe implements PipeTransform {
   transform(value: string, locale: string): moment.Moment {
-    return moment(value).locale(locale);
+    return momentConstructor(value).locale(locale);
   }
 }

src/parse.pipe.ts

@@ -1,12 +1,11 @@
 import { Pipe, PipeTransform } from '@angular/core';
 import * as moment from 'moment';
 
-// under systemjs, moment is actually exported as the default export, so we account for that
-const momentConstructor: (value?: any) => moment.Moment = (<any>moment).default || moment;
+const momentConstructor = moment;
 
 @Pipe({ name: 'amParse' })
 export class ParsePipe implements PipeTransform {
   transform(value: string, format: string): moment.Moment {
-    return moment(value, format);
+    return momentConstructor(value, format);
   }
 }

src/subtract.pipe.ts

@@ -1,14 +1,16 @@
 /* ngx-moment (c) 2015, 2016 Uri Shaked / MIT Licence */
 
-import {Pipe, ChangeDetectorRef, PipeTransform} from '@angular/core';
+import {Pipe, PipeTransform} from '@angular/core';
 import * as moment from 'moment';
 
+const momentConstructor = moment;
+
 @Pipe({ name: 'amSubtract' })
 export class SubtractPipe implements PipeTransform {
     transform(value: any, amount: moment.DurationInputArg1, unit?: moment.DurationInputArg2): any {
         if (typeof amount === 'undefined' || (typeof amount === 'number' && typeof unit === 'undefined')) {
             throw new Error('SubtractPipe: missing required arguments');
         }
-        return moment(value).subtract(amount, unit);
+        return momentConstructor(value).subtract(amount, unit);
     }
 }

src/time-ago.pipe.spec.ts

@@ -7,9 +7,6 @@ import 'moment/min/locales';
 
 declare var global: any;
 
-// under systemjs, moment is actually exported as the default export, so we account for that
-const momentConstructor: (value?: any) => moment.Moment = (<any>moment).default || moment;
-
 class NgZoneMock {
   runOutsideAngular(fn: Function) {
     return fn();

src/time-ago.pipe.ts

@@ -3,8 +3,7 @@
 import {Pipe, ChangeDetectorRef, PipeTransform, OnDestroy, NgZone} from '@angular/core';
 import * as moment from 'moment';
 
-// under systemjs, moment is actually exported as the default export, so we account for that
-const momentConstructor: (value?: any) => moment.Moment = (<any>moment).default || moment;
+const momentConstructor = moment;
 
 @Pipe({name: 'amTimeAgo', pure: false})
 export class TimeAgoPipe implements PipeTransform, OnDestroy {

src/utc.pipe.ts

@@ -1,12 +1,11 @@
 import { Pipe, PipeTransform } from '@angular/core';
 import * as moment from 'moment';
 
-// under systemjs, moment is actually exported as the default export, so we account for that
-const momentConstructor: (value?: any) => moment.Moment = (<any>moment).default || moment;
+const momentConstructor = moment;
 
 @Pipe({ name: 'amUtc' })
 export class UtcPipe implements PipeTransform {
   transform(value: Date | moment.Moment | string | number): moment.Moment {
-    return moment(value).utc();
+    return momentConstructor(value).utc();
   }
 }

tsconfig.json

@@ -1,7 +1,6 @@
 {
   "version": "2.0.3",
   "compilerOptions": {
-    "noImplicitAny": true,
     "module": "commonjs",
     "target": "es5",
     "lib": [
@@ -10,41 +9,15 @@
     ],
     "emitDecoratorMetadata": true,
     "experimentalDecorators": true,
+    "noImplicitAny": true,
+    "noImplicitReturns": true,
+    "noImplicitThis": true,
+    "noUnusedLocals": true,
     "sourceMap": true,
     "inlineSources": true,
     "declaration": true
   },
-  "files": [
-    "src/add.pipe.ts",
-    "src/add.pipe.spec.ts",
-    "src/calendar.pipe.ts",
-    "src/calendar.pipe.spec.ts",
-    "src/date-format.pipe.ts",
-    "src/date-format.pipe.spec.ts",
-    "src/difference.pipe.ts",
-    "src/difference.pipe.spec.ts",
-    "src/duration.pipe.ts",
-    "src/duration.pipe.spec.ts",
-    "src/from-unix.pipe.ts",
-    "src/from-unix.pipe.spec.ts",
-    "src/index.ts",
-    "src/parse.pipe.ts",
-    "src/parse.pipe.spec.ts",
-    "src/moment.module.ts",
-    "src/subtract.pipe.ts",
-    "src/subtract.pipe.spec.ts",
-    "src/time-ago.pipe.ts",
-    "src/time-ago.pipe.spec.ts",
-    "src/utc.pipe.ts",
-    "src/utc.pipe.spec.ts",
-    "src/from-utc.pipe.ts",
-    "src/from-utc.pipe.spec.ts",
-    "src/local.pipe.ts",
-    "src/local.pipe.spec.ts",
-    "src/locale.pipe.ts",
-    "src/locale.pipe.spec.ts"
-  ],
-  "angularCompilerOptions": {
-    "genDir": "compiled"
-  }
+  "include": [
+    "src/*.ts"
+  ]
 }