build: upgrade to Angular and Material v7 and add strict flags (#855)

* Upgrade to Angular v7
* Upgrade to Angular Material v7
* Add nostrictPropertyInitialization flag
* Add noUnusedLocals flag and remove tslint option
* Add noImplicitThis flag
* Add noImplicitAny flag
* Enable strictMetadataEmit flag
* Various refactors to support the above flags
* Switch to yarn

Fixes #851

.circleci/config.yml

@@ -22,16 +22,16 @@ jobs:
       # Download and cache dependencies
       - restore_cache:
           keys:
-          - v1-dependencies-{{ checksum "package.json" }}
+          - v1-dependencies-{{ checksum "yarn.lock" }}
           # fallback to using the latest cache if no exact match is found
           - v1-dependencies-
 
-      - run: npm install
+      - run: yarn install --frozen-lockfile --non-interactive
 
       - save_cache:
           paths:
             - node_modules
-          key: v1-dependencies-{{ checksum "package.json" }}
+          key: v1-dependencies-{{ checksum "yarn.lock" }}
 
       # run the build!
       - run: npm run lib:build

.travis.yml

@@ -50,7 +50,7 @@ before_install:
   - source ./scripts/ci/env.sh
 
 install:
-  - npm install
+  - yarn install --frozen-lockfile --non-interactive
 
 before_script:
   - mkdir -p $LOGS_DIR
@@ -59,5 +59,6 @@ script:
   - ./scripts/ci/travis-script.sh
 
 cache:
+  yarn: true
   directories:
     - ./node_modules/

package.json

@@ -26,7 +26,7 @@
     "universal:serve": "gulp universal:serve"
   },
   "version": "6.0.0-beta.18",
-  "requiredAngularVersion": ">=6.0.0 <7.0.0",
+  "requiredAngularVersion": ">=7.0.0-rc.0",
   "dependencies": {
     "@angular/cdk": "^6.4.7",
     "@angular/common": "^6.1.6",
@@ -66,7 +66,7 @@
     "cli-color": "^1.3.0",
     "dgeni": "^0.4.10",
     "dgeni-packages": "^0.26.7",
-    "firebase": "^4.13.1",
+    "firebase": "^5.5.2",
     "firebase-admin": "^5.13.1",
     "firebase-tools": "^4.2.1",
     "fs-extra": "^3.0.1",

src/lib/core/base/base-adapter.spec.ts

@@ -12,7 +12,7 @@ import {MediaMonitor} from '../media-monitor/media-monitor';
 import {StyleUtils} from '../style-utils/style-utils';
 
 describe('BaseDirectiveAdapter class', () => {
-  let component;
+  let component: any;
   beforeEach(() => {
     component = new BaseDirectiveAdapter('', {} as MediaMonitor, {} as ElementRef, {} as StyleUtils); // tslint:disable-line:max-line-length
   });

src/lib/core/base/base-adapter.ts

@@ -40,7 +40,7 @@ export class BaseDirectiveAdapter extends BaseDirective {
    * @see BaseDirective._mqActivation
    */
   get mqActivation(): ResponsiveActivation {
-    return this._mqActivation;
+    return this._mqActivation!;
   }
 
   /**
@@ -64,7 +64,7 @@ export class BaseDirectiveAdapter extends BaseDirective {
   /**
    * @see BaseDirective._queryInput
    */
-  queryInput(key) {
+  queryInput(key: string | null) {
     return key ? this._queryInput(key) : undefined;
   }
 

src/lib/core/base/base.ts

@@ -108,7 +108,7 @@ export abstract class BaseDirective implements OnDestroy, OnChanges {
   }
 
   /** Access the current value (if any) of the @Input property */
-  protected _queryInput(key) {
+  protected _queryInput(key: string) {
     return this._inputMap[key];
   }
 
@@ -218,19 +218,19 @@ export abstract class BaseDirective implements OnDestroy, OnChanges {
   }
 
   /** Fast validator for presence of attribute on the host element */
-  protected hasKeyValue(key) {
-    return this._mqActivation.hasKeyValue(key);
+  protected hasKeyValue(key: string) {
+    return this._mqActivation!.hasKeyValue(key);
   }
 
   protected get hasInitialized() {
     return this._hasInitialized;
   }
 
   /** MediaQuery Activation Tracker */
-  protected _mqActivation: ResponsiveActivation;
+  protected _mqActivation?: ResponsiveActivation;
 
   /** Dictionary of input keys with associated values */
-  protected _inputMap = {};
+  protected _inputMap: {[key: string]: any} = {};
 
   /**
    * Has the `ngOnInit()` method fired