style(linting): add global tslint rules

jest/tests-setup.ts

@@ -1,10 +1,14 @@
+// TODO: Should be able to remove this once dev dependencies are hoisted to the workspace
+// tslint:disable:no-implicit-dependencies
 import 'core-js/es6/reflect';
 import 'core-js/es7/reflect';
 import 'zone.js';
-import 'zone.js/dist/proxy';
-import 'zone.js/dist/sync-test';
 import 'zone.js/dist/async-test';
 import 'zone.js/dist/fake-async-test';
+import 'zone.js/dist/proxy';
+import 'zone.js/dist/sync-test';
+// This must be loaded in after ZoneJS
+// tslint:disable-next-line:ordered-imports
 import 'jest-zone-patch';
 
 import { TestBed } from '@angular/core/testing';

package.json

@@ -9,7 +9,7 @@
     "clean:workspace-deps": "rimraf node_modules",
     "clean:package-deps": "rimraf packages/*/node_modules",
     "clean:coverage": "rimraf coverage",
-    "lint": "npm-run-all -p lint:*",
+    "lint": "tslint -p tsconfig.json",
     "lint:packages": "lerna run lint",
     "lint:prettier": "prettier -l \"**/*.*(ts|js|css|scss|json|md)\"",
     "test": "jest --coverage",
@@ -31,14 +31,17 @@
     "@commitlint/config-conventional": "7.0.1",
     "@commitlint/prompt-cli": "7.0.0",
     "@types/jest": "23.1.6",
+    "codelyzer": "4.4.2",
     "husky": "0.14.3",
     "jest": "23.4.1",
     "jest-junit": "5.1.0",
     "jest-preset-angular": "5.2.3",
+    "jest-zone-patch": "0.0.8",
     "lerna": "2.11.0",
     "lint-staged": "7.2.0",
     "npm-run-all": "4.1.3",
     "prettier": "1.13.7",
-    "rimraf": "2.6.2"
+    "rimraf": "2.6.2",
+    "tslint": "5.11.0"
   }
 }

packages/example-app/package.json

@@ -37,10 +37,8 @@
     "@angular/compiler-cli": "^4.1.0",
     "@types/node": "~6.0.71",
     "@types/redux-logger": "^3.0.0",
-    "codelyzer": "~3.0.1",
     "protractor": "~5.1.1",
     "ts-node": "~3.0.2",
-    "tslint": "~5.1.0",
     "typescript": "^2.4.1"
   }
 }

packages/example-app/src/app/animals/animal-list/component.spec.ts

@@ -1,9 +1,9 @@
 import { Component, Input } from '@angular/core';
-import { TestBed, async } from '@angular/core/testing';
+import { async, TestBed } from '@angular/core/testing';
 import { By } from '@angular/platform-browser';
-import { AnimalListComponent } from './component';
 import { CoreModule } from '../../core/module';
 import { AnimalType } from '../model';
+import { AnimalListComponent } from './component';
 
 @Component({ selector: 'zoo-animal', template: '' })
 class MockAnimalComponent {
@@ -19,7 +19,7 @@ xdescribe('AnimalListComponent', () => {
     }).compileComponents();
   }));
 
-  it(`should have as title 'Welcome to the Zoo'`, async(() => {
+  it("should have as title 'Welcome to the Zoo'", async(() => {
     const fixture = TestBed.createComponent(AnimalListComponent);
     const animalList = fixture.debugElement.componentInstance;
 

packages/example-app/src/app/animals/animal-list/component.ts

@@ -1,6 +1,6 @@
-import { Component, Input, ChangeDetectionStrategy } from '@angular/core';
+import { ChangeDetectionStrategy, Component, Input } from '@angular/core';
 import { Observable } from 'rxjs/Observable';
-import { IAnimal } from '../model';
+import { Animal } from '../model';
 
 @Component({
   selector: 'zoo-animal-list',
@@ -11,14 +11,14 @@ import { IAnimal } from '../model';
 export class AnimalListComponent {
   @Input() animalsName: string;
   @Input() animalType: string;
-  @Input() animals: Observable<IAnimal[]>;
+  @Input() animals: Observable<Animal[]>;
   @Input() loading: Observable<boolean>;
   @Input() error: Observable<any>;
 
   // Since we're observing an array of items, we need to set up a 'trackBy'
   // parameter so Angular doesn't tear down and rebuild the list's DOM every
   // time there's an update.
-  getKey(_, animal: IAnimal) {
+  getKey(_, animal: Animal) {
     return animal.id;
   }
 }

packages/example-app/src/app/animals/animal/component.spec.ts

@@ -1,11 +1,11 @@
-import { async, TestBed } from '@angular/core/testing';
 import {
-  NgReduxTestingModule,
   MockNgRedux,
+  NgReduxTestingModule,
 } from '@angular-redux/store/testing';
-import { AnimalComponent } from './component';
-import { CoreModule } from '../../core/module';
+import { async, TestBed } from '@angular/core/testing';
 import 'rxjs/add/operator/toArray';
+import { CoreModule } from '../../core/module';
+import { AnimalComponent } from './component';
 
 xdescribe('AnimalComponent', () => {
   let fixture;

packages/example-app/src/app/animals/animal/component.ts

@@ -1,11 +1,11 @@
-import { Component, Input, ChangeDetectionStrategy } from '@angular/core';
 import { dispatch, select, select$, WithSubStore } from '@angular-redux/store';
+import { ChangeDetectionStrategy, Component, Input } from '@angular/core';
 import { Observable } from 'rxjs/Observable';
 
+import { Animal } from '../model';
 import { animalComponentReducer } from './reducers';
-import { IAnimal } from '../model';
 
-export const toSubTotal = (obs$: Observable<IAnimal>): Observable<number> =>
+export const toSubTotal = (obs$: Observable<Animal>): Observable<number> =>
   obs$.map(s => s.ticketPrice * s.tickets);
 
 /**

packages/example-app/src/app/animals/animal/reducers.ts

@@ -1,4 +1,4 @@
-import { Reducer, Action } from 'redux';
+import { Action, Reducer } from 'redux';
 import { AnimalComponent } from './component';
 
 export const ticketsReducer: Reducer<number> = (

packages/example-app/src/app/animals/api/actions.ts

@@ -1,10 +1,10 @@
-import { Injectable } from '@angular/core';
 import { dispatch } from '@angular-redux/store';
+import { Injectable } from '@angular/core';
 import { FluxStandardAction } from 'flux-standard-action';
-import { IAnimal, AnimalType } from '../model';
+import { Animal, AnimalType } from '../model';
 
 // Flux-standard-action gives us stronger typing of our actions.
-type Payload = IAnimal[];
+type Payload = Animal[];
 interface MetaData {
   animalType: AnimalType;
 }

packages/example-app/src/app/animals/api/epics.ts

@@ -1,19 +1,19 @@
 import { Injectable } from '@angular/core';
-import { Epic, createEpicMiddleware } from 'redux-observable';
-import { of } from 'rxjs/observable/of';
+import { createEpicMiddleware, Epic } from 'redux-observable';
 import 'rxjs/add/operator/catch';
-import 'rxjs/add/operator/map';
 import 'rxjs/add/operator/do';
+import 'rxjs/add/operator/map';
 import 'rxjs/add/operator/startWith';
+import { of } from 'rxjs/observable/of';
 
-import { IAppState } from '../../store/model';
+import { AppState } from '../../store/model';
 import { AnimalType } from '../model';
 import { AnimalAPIAction, AnimalAPIActions } from './actions';
 import { AnimalAPIService } from './service';
 
 const animalsNotAlreadyFetched = (
   animalType: AnimalType,
-  state: IAppState,
+  state: AppState,
 ): boolean =>
   !(
     state[animalType] &&
@@ -32,13 +32,13 @@ export class AnimalAPIEpics {
     private actions: AnimalAPIActions,
   ) {}
 
-  public createEpic(animalType: AnimalType) {
+  createEpic(animalType: AnimalType) {
     return createEpicMiddleware(this.createLoadAnimalEpic(animalType));
   }
 
   private createLoadAnimalEpic(
     animalType: AnimalType,
-  ): Epic<AnimalAPIAction, IAppState> {
+  ): Epic<AnimalAPIAction, AppState> {
     return (action$, store) =>
       action$
         .ofType(AnimalAPIActions.LOAD_ANIMALS)

packages/example-app/src/app/animals/api/reducer.ts

@@ -1,9 +1,9 @@
-import { AnimalAPIAction, AnimalAPIActions } from './actions';
-import { IAnimalList, AnimalType } from '../model';
 import { indexBy, prop } from 'ramda';
 import { Action } from 'redux';
+import { AnimalList, AnimalType } from '../model';
+import { AnimalAPIAction, AnimalAPIActions } from './actions';
 
-const INITIAL_STATE: IAnimalList = {
+const INITIAL_STATE: AnimalList = {
   items: {},
   loading: false,
   error: null,
@@ -13,9 +13,9 @@ const INITIAL_STATE: IAnimalList = {
 // that only responds to actions for that particular animal type.
 export function createAnimalAPIReducer(animalType: AnimalType) {
   return function animalReducer(
-    state: IAnimalList = INITIAL_STATE,
+    state: AnimalList = INITIAL_STATE,
     a: Action,
-  ): IAnimalList {
+  ): AnimalList {
     const action = a as AnimalAPIAction;
     if (!action.meta || action.meta.animalType !== animalType) {
       return state;

packages/example-app/src/app/animals/api/service.ts

@@ -1,10 +1,10 @@
 import { Injectable } from '@angular/core';
 import { Http } from '@angular/http';
-import { Observable } from 'rxjs/Observable';
-import 'rxjs/add/operator/map';
 import 'rxjs/add/observable/of';
+import 'rxjs/add/operator/map';
+import { Observable } from 'rxjs/Observable';
 
-import { ANIMAL_TYPES, AnimalType, IAnimal, fromServer } from '../model';
+import { Animal, ANIMAL_TYPES, AnimalType, fromServer } from '../model';
 
 // A fake API on the internets.
 const URLS = {
@@ -16,7 +16,7 @@ const URLS = {
 export class AnimalAPIService {
   constructor(private http: Http) {}
 
-  getAll = (animalType: AnimalType): Observable<IAnimal[]> =>
+  getAll = (animalType: AnimalType): Observable<Animal[]> =>
     this.http
       .get(URLS[animalType])
       .map(resp => resp.json())

packages/example-app/src/app/animals/model.ts

@@ -6,21 +6,21 @@ export const ANIMAL_TYPES = {
 // TODO: is there a way to improve this?
 export type AnimalType = string;
 
-export interface IAnimal {
+export interface Animal {
   id: string;
   animalType: AnimalType;
   name: string;
   ticketPrice: number;
   tickets: number;
 }
 
-export interface IAnimalList {
+export interface AnimalList {
   items: {};
   loading: boolean;
   error: any;
 }
 
-export const fromServer = (record: any): IAnimal => ({
+export const fromServer = (record: any): Animal => ({
   id: record.name.toLowerCase(),
   animalType: record.animalType,
   name: record.name,

packages/example-app/src/app/animals/module.ts

@@ -1,12 +1,12 @@
-import { NgModule } from '@angular/core';
 import { CommonModule } from '@angular/common';
+import { NgModule } from '@angular/core';
 
 import { CoreModule } from '../core/module';
+import { StoreModule } from '../store/module';
 import { AnimalListComponent } from './animal-list/component';
 import { AnimalAPIActions } from './api/actions';
 import { AnimalAPIEpics } from './api/epics';
 import { AnimalAPIService } from './api/service';
-import { StoreModule } from '../store/module';
 
 import { AnimalComponent } from './animal/component';
 

packages/example-app/src/app/component.html

@@ -1,5 +1,5 @@
 <h1>
-  {{title}}
+  {{ title }}
 </h1>
 <nav>
   <a routerLink="/elephants" routerLinkActive="active">Elephants</a>

packages/example-app/src/app/component.spec.ts

@@ -1,4 +1,4 @@
-import { TestBed, async } from '@angular/core/testing';
+import { async, TestBed } from '@angular/core/testing';
 import { RouterTestingModule } from '@angular/router/testing';
 import { AppComponent } from './component';
 
@@ -10,7 +10,7 @@ xdescribe('AppComponent', () => {
     }).compileComponents();
   }));
 
-  it(`should have as title 'Welcome to the Zoo'`, async(() => {
+  it("should have as title 'Welcome to the Zoo'", async(() => {
     const fixture = TestBed.createComponent(AppComponent);
     const app = fixture.debugElement.componentInstance;
     expect(app.title).toEqual('Welcome to the Zoo');

packages/example-app/src/app/component.ts

@@ -1,4 +1,4 @@
-import { Component, ChangeDetectionStrategy } from '@angular/core';
+import { ChangeDetectionStrategy, Component } from '@angular/core';
 
 @Component({
   selector: 'zoo-root',

packages/example-app/src/app/core/counter/component.ts

@@ -1,9 +1,9 @@
 import {
+  ChangeDetectionStrategy,
   Component,
+  EventEmitter,
   Input,
   Output,
-  ChangeDetectionStrategy,
-  EventEmitter,
 } from '@angular/core';
 
 @Component({

packages/example-app/src/app/core/error-well/component.ts

@@ -1,4 +1,4 @@
-import { Input, Component, ChangeDetectionStrategy } from '@angular/core';
+import { ChangeDetectionStrategy, Component, Input } from '@angular/core';
 import { Observable } from 'rxjs/Observable';
 
 @Component({

packages/example-app/src/app/core/module.ts

@@ -1,9 +1,9 @@
-import { NgModule } from '@angular/core';
 import { CommonModule } from '@angular/common';
+import { NgModule } from '@angular/core';
 
-import { SpinnerComponent } from './spinner/component';
-import { ErrorWellComponent } from './error-well/component';
 import { CounterComponent } from './counter/component';
+import { ErrorWellComponent } from './error-well/component';
+import { SpinnerComponent } from './spinner/component';
 
 @NgModule({
   declarations: [SpinnerComponent, ErrorWellComponent, CounterComponent],

packages/example-app/src/app/elephants/module.ts

@@ -1,10 +1,10 @@
-import { NgModule } from '@angular/core';
 import { CommonModule } from '@angular/common';
+import { NgModule } from '@angular/core';
 
-import { ElephantPageComponent } from './page';
 import { AnimalModule } from '../animals/module';
 import { CoreModule } from '../core/module';
 import { StoreModule } from '../store/module';
+import { ElephantPageComponent } from './page';
 
 @NgModule({
   declarations: [ElephantPageComponent],

packages/example-app/src/app/elephants/page.spec.ts

@@ -1,19 +1,19 @@
-import { TestBed } from '@angular/core/testing';
 import {
-  NgReduxTestingModule,
   MockNgRedux,
+  NgReduxTestingModule,
 } from '@angular-redux/store/testing';
+import { TestBed } from '@angular/core/testing';
 
 import { Component, Input } from '@angular/core';
 
-import { Observable } from 'rxjs/Observable';
 import 'rxjs/add/observable/of';
-import 'rxjs/add/operator/toArray';
 import 'rxjs/add/operator/do';
+import 'rxjs/add/operator/toArray';
+import { Observable } from 'rxjs/Observable';
 
-import { ElephantPageComponent } from './page';
 import { AnimalAPIActions } from '../animals/api/actions';
 import { ANIMAL_TYPES } from '../animals/model';
+import { ElephantPageComponent } from './page';
 
 @Component({
   selector: 'zoo-animal-list',