style(linting): add global tslint rules (#35)

Adds tslint rules to be run across the workspace based off of the latest preset with some tweaks to taste

BREAKING CHANGE:
- ConnectArray has been renamed to ConnectArrayDirective
- ReactiveConnect has been renamed to ReactiveConnectDirective
- Connect has been renamed to ConnectDirective
- interfaces with an "I" prefix have had that prefix removed (e.g "IAppStore" -> "AppStore")
closes #4.

Author: smithad15

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/circle.yml

@@ -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/package.json

@@ -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.spec.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-list/component.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.spec.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/component.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/animal/reducers.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;
 }