build: bump to tslint 5.8.0 and fix failures (#7943)

Updates to `tslint@5.8.0` and fixes a few previously-uncaught failures.

package.json

@@ -116,7 +116,7 @@
     "stylelint": "^7.12.0",
     "ts-node": "^3.0.4",
     "tsconfig-paths": "^2.2.0",
-    "tslint": "^5.7.0",
+    "tslint": "^5.8.0",
     "tsutils": "^2.6.0",
     "typescript": "~2.4.2",
     "uglify-js": "^2.8.14",

src/lib/core/option/option.ts

@@ -110,7 +110,7 @@ export class MatOption {
     private _element: ElementRef,
     private _changeDetectorRef: ChangeDetectorRef,
     @Optional() @Inject(MAT_OPTION_PARENT_COMPONENT) private _parent: MatOptionParentComponent,
-    @Optional() public readonly group: MatOptgroup) {}
+    @Optional() readonly group: MatOptgroup) {}
 
   /**
    * Whether or not the option is currently active and ready to be selected.

src/lib/dialog/dialog-ref.ts

@@ -44,7 +44,7 @@ export class MatDialogRef<T> {
   constructor(
     private _overlayRef: OverlayRef,
     private _containerInstance: MatDialogContainer,
-    public readonly id: string = `mat-dialog-${uniqueId++}`) {
+    readonly id: string = `mat-dialog-${uniqueId++}`) {
 
     // Emit when opening animation completes
     RxChain.from(_containerInstance._animationStateChanged)

tools/package-tools/build-package.ts

@@ -58,7 +58,7 @@ export class BuildPackage {
   }
   private _secondaryEntryPoints: string[];
 
-  constructor(public readonly name: string, public readonly dependencies: BuildPackage[] = []) {
+  constructor(readonly name: string, readonly dependencies: BuildPackage[] = []) {
     this.sourceDir = join(packagesDir, name);
     this.outputDir = join(outputDir, 'packages', name);
     this.esm5OutputDir = join(outputDir, 'packages', name, 'esm5');