build: enable no-unused-variable lint rule

devversion · devversion · commit 80681c40c073 · 2017-06-26T18:34:42.000+02:00
* Now TSLint starts reporting unused variables again (Rule has been un-deprecated and now uses the type checker
* In favor of linting the whole project the tsconfig file inside of `src/` needs to live in the project root.
diff --git a/e2e/components/checkbox-e2e.spec.ts b/e2e/components/checkbox-e2e.spec.ts
@@ -10,7 +10,6 @@ describe('checkbox', () => {
     it('should be checked when clicked, and unchecked when clicked again', async () => {
       let checkboxEl = element(by.id('test-checkbox'));
       let inputEl = element(by.css('input[id=input-test-checkbox]'));
-      let checked: string;
 
       screenshot('start');
       checkboxEl.click();
diff --git a/e2e/components/icon-e2e.spec.ts b/e2e/components/icon-e2e.spec.ts
@@ -1,5 +1,4 @@
 import {browser, by, element} from 'protractor';
-import {screenshot} from '../screenshot';
 
 
 describe('icon', () => {
diff --git a/e2e/components/menu-e2e.spec.ts b/e2e/components/menu-e2e.spec.ts
@@ -1,4 +1,4 @@
-import {Key, protractor, browser, by, element, ElementFinder} from 'protractor';
+import {Key, protractor, browser, by, element} from 'protractor';
 import {screenshot} from '../screenshot';
 import {
   expectToExist,
diff --git a/src/demo-app/overlay/overlay-demo.ts b/src/demo-app/overlay/overlay-demo.ts
@@ -11,6 +11,9 @@ import {
     OverlayState,
     OverlayOrigin,
     ComponentPortal,
+    // This import is only used to define a generic type. The current TypeScript version incorrectly
+    // considers such imports as unused (https://github.com/Microsoft/TypeScript/issues/14953)
+    // tslint:disable-next-line:no-unused-variable
     Portal,
     TemplatePortalDirective,
 } from '@angular/material';
diff --git a/src/lib/autocomplete/autocomplete.spec.ts b/src/lib/autocomplete/autocomplete.spec.ts
@@ -30,7 +30,7 @@ import {dispatchFakeEvent} from '../core/testing/dispatch-events';
 import {createKeyboardEvent} from '../core/testing/event-objects';
 import {typeInElement} from '../core/testing/type-in-element';
 import {ScrollDispatcher} from '../core/overlay/scroll/scroll-dispatcher';
-import {RxChain, map, startWith, filter} from '../core/rxjs/index';
+import {RxChain, map, startWith} from '../core/rxjs/index';
 
 
 describe('MdAutocomplete', () => {
diff --git a/src/lib/chips/chip-list.ts b/src/lib/chips/chip-list.ts
@@ -19,9 +19,9 @@ import {
 
 import {MdChip} from './chip';
 import {FocusKeyManager} from '../core/a11y/focus-key-manager';
-import {SPACE, LEFT_ARROW, RIGHT_ARROW, TAB} from '../core/keyboard/keycodes';
-import {coerceBooleanProperty} from '@angular/cdk';
+import {SPACE, LEFT_ARROW, RIGHT_ARROW} from '../core/keyboard/keycodes';
 import {Subscription} from 'rxjs/Subscription';
+import {coerceBooleanProperty} from '@angular/cdk';
 
 /**
  * A material design chips component (named ChipList for it's similarity to the List component).
diff --git a/src/lib/core/a11y/activedescendant-key-manager.ts b/src/lib/core/a11y/activedescendant-key-manager.ts
@@ -6,7 +6,6 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {QueryList} from '@angular/core';
 import {ListKeyManager, CanDisable} from './list-key-manager';
 
 /**
diff --git a/src/lib/core/bidi/index.ts b/src/lib/core/bidi/index.ts
@@ -9,7 +9,7 @@
 import {NgModule} from '@angular/core';
 import {DOCUMENT} from '@angular/platform-browser';
 import {Dir} from './dir';
-import {DIR_DOCUMENT, Directionality, DIRECTIONALITY_PROVIDER} from './directionality';
+import {DIR_DOCUMENT, Directionality} from './directionality';
 
 
 export {
diff --git a/src/lib/core/data-table/cell.ts b/src/lib/core/data-table/cell.ts
@@ -47,10 +47,8 @@ export class CdkColumnDef {
   },
 })
 export class CdkHeaderCell {
-  constructor(private columnDef: CdkColumnDef,
-              private elementRef: ElementRef,
-              private renderer: Renderer2) {
-    this.renderer.addClass(elementRef.nativeElement, `cdk-column-${columnDef.name}`);
+  constructor(columnDef: CdkColumnDef, elementRef: ElementRef, renderer: Renderer2) {
+    renderer.addClass(elementRef.nativeElement, `cdk-column-${columnDef.name}`);
   }
 }
 
@@ -63,9 +61,7 @@ export class CdkHeaderCell {
   },
 })
 export class CdkCell {
-  constructor(private columnDef: CdkColumnDef,
-              private elementRef: ElementRef,
-              private renderer: Renderer2) {
-    this.renderer.addClass(elementRef.nativeElement, `cdk-column-${columnDef.name}`);
+  constructor(columnDef: CdkColumnDef, elementRef: ElementRef, renderer: Renderer2) {
+    renderer.addClass(elementRef.nativeElement, `cdk-column-${columnDef.name}`);
   }
 }
diff --git a/src/lib/core/datetime/native-date-adapter.spec.ts b/src/lib/core/datetime/native-date-adapter.spec.ts
@@ -1,16 +1,9 @@
 import {NativeDateAdapter} from './native-date-adapter';
 import {Platform} from '../platform/index';
-
+import {DEC, FEB, JAN, MAR} from '../testing/month-constants';
 
 const SUPPORTS_INTL = typeof Intl != 'undefined';
 
-
-// When constructing a Date, the month is zero-based. This can be confusing, since people are
-// used to seeing them one-based. So we create these aliases to make reading the tests easier.
-const JAN = 0, FEB = 1, MAR = 2, APR = 3, MAY = 4, JUN = 5, JUL = 6, AUG = 7, SEP = 8, OCT = 9,
-      NOV = 10, DEC = 11;
-
-
 describe('NativeDateAdapter', () => {
   let adapter;
   let platform;
diff --git a/src/lib/core/option/optgroup.ts b/src/lib/core/option/optgroup.ts
@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {Component, ViewEncapsulation, ContentChildren, QueryList, Input} from '@angular/core';
+import {Component, ViewEncapsulation, Input} from '@angular/core';
 import {mixinDisabled, CanDisable} from '../common-behaviors/disabled';
 
 // Boilerplate for applying mixins to MdOptgroup.
diff --git a/src/lib/core/option/option.ts b/src/lib/core/option/option.ts
@@ -12,7 +12,6 @@ import {
   EventEmitter,
   Input,
   Output,
-  NgModule,
   ViewEncapsulation,
   Inject,
   Optional,
diff --git a/src/lib/core/overlay/overlay-directives.ts b/src/lib/core/overlay/overlay-directives.ts
@@ -26,11 +26,13 @@ import {TemplatePortal} from '../portal/portal';
 import {OverlayState} from './overlay-state';
 import {
     ConnectionPositionPair,
+    // This import is only used to define a generic type. The current TypeScript version incorrectly
+    // considers such imports as unused (https://github.com/Microsoft/TypeScript/issues/14953)
+    // tslint:disable-next-line:no-unused-variable
     ConnectedOverlayPositionChange
 } from './position/connected-position';
 import {ConnectedPositionStrategy} from './position/connected-position-strategy';
 import {Directionality, Direction} from '../bidi/index';
-import {Scrollable} from './scroll/scrollable';
 import {ScrollStrategy} from './scroll/scroll-strategy';
 import {coerceBooleanProperty} from '@angular/cdk';
 import {ESCAPE} from '../keyboard/keycodes';
diff --git a/src/lib/core/overlay/overlay.spec.ts b/src/lib/core/overlay/overlay.spec.ts
@@ -9,9 +9,7 @@ import {
   OverlayContainer,
   Overlay,
   PositionStrategy,
-  ViewportRuler,
   ScrollStrategy,
-  ScrollDispatcher,
 } from './index';
 
 
@@ -336,7 +334,6 @@ describe('Overlay', () => {
       overlayRef.attach(componentPortal);
       viewContainerFixture.detectChanges();
 
-      let backdrop = overlayContainerElement.querySelector('.cdk-overlay-backdrop') as HTMLElement;
       let completeHandler = jasmine.createSpy('backdrop complete handler');
 
       overlayRef.backdropClick().subscribe(undefined, undefined, completeHandler);
diff --git a/src/lib/core/overlay/scroll/scroll-strategy-options.ts b/src/lib/core/overlay/scroll/scroll-strategy-options.ts
@@ -7,7 +7,6 @@
  */
 
 import {Injectable} from '@angular/core';
-import {ScrollStrategy} from './scroll-strategy';
 import {CloseScrollStrategy} from './close-scroll-strategy';
 import {NoopScrollStrategy} from './noop-scroll-strategy';
 import {BlockScrollStrategy} from './block-scroll-strategy';
diff --git a/src/lib/core/rxjs/rx-operators.ts b/src/lib/core/rxjs/rx-operators.ts
@@ -9,7 +9,7 @@
 import {Observable, ObservableInput} from 'rxjs/Observable';
 import {PartialObserver} from 'rxjs/Observer';
 import {Subscription} from 'rxjs/Subscription';
-import {Scheduler, IScheduler} from 'rxjs/Scheduler';
+import {IScheduler} from 'rxjs/Scheduler';
 import {_finally as _finallyOperator} from 'rxjs/operator/finally';
 import {_catch as _catchOperator} from 'rxjs/operator/catch';
 import {_do as _doOperator} from 'rxjs/operator/do';
diff --git a/src/lib/core/testing/month-constants.ts b/src/lib/core/testing/month-constants.ts
@@ -0,0 +1,14 @@
+/**
+ * @license
+ * Copyright Google Inc. All Rights Reserved.
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.io/license
+ */
+
+/**
+ * When constructing a Date, the month is zero-based. This can be confusing, since people are
+ * used to seeing them one-based. So we create these aliases to make writing the tests easier.
+ */
+export const JAN = 0, FEB = 1, MAR = 2, APR = 3, MAY = 4, JUN = 5, JUL = 6, AUG = 7, SEP = 8,
+    OCT = 9, NOV = 10, DEC = 11;
diff --git a/src/lib/datepicker/calendar.spec.ts b/src/lib/datepicker/calendar.spec.ts
@@ -25,13 +25,7 @@ import {MdDatepickerIntl} from './datepicker-intl';
 import {MdNativeDateModule} from '../core/datetime/index';
 import {NoConflictStyleCompatibilityMode} from '../core';
 import {MdButtonModule} from '../button/index';
-
-
-// When constructing a Date, the month is zero-based. This can be confusing, since people are
-// used to seeing them one-based. So we create these aliases to make reading the tests easier.
-const JAN = 0, FEB = 1, MAR = 2, APR = 3, MAY = 4, JUN = 5, JUL = 6, AUG = 7, SEP = 8, OCT = 9,
-      NOV = 10, DEC = 11;
-
+import {AUG, DEC, FEB, JAN, JUL, NOV, MAR, MAY, JUN, SEP} from '../core/testing/month-constants';
 
 describe('MdCalendar', () => {
   beforeEach(async(() => {
diff --git a/src/lib/datepicker/datepicker.spec.ts b/src/lib/datepicker/datepicker.spec.ts
@@ -14,13 +14,7 @@ import {
   dispatchMouseEvent,
   dispatchKeyboardEvent,
 } from '../core/testing/dispatch-events';
-
-
-// When constructing a Date, the month is zero-based. This can be confusing, since people are
-// used to seeing them one-based. So we create these aliases to make reading the tests easier.
-const JAN = 0, FEB = 1, MAR = 2, APR = 3, MAY = 4, JUN = 5, JUL = 6, AUG = 7, SEP = 8, OCT = 9,
-      NOV = 10, DEC = 11;
-
+import {DEC, JAN} from '../core/testing/month-constants';
 
 describe('MdDatepicker', () => {
   describe('with MdNativeDateModule', () => {
diff --git a/src/lib/datepicker/datepicker.ts b/src/lib/datepicker/datepicker.ts
@@ -33,7 +33,6 @@ import {MdDialogRef} from '../dialog/dialog-ref';
 import {PositionStrategy} from '../core/overlay/position/position-strategy';
 import {MdDatepickerInput} from './datepicker-input';
 import {Subscription} from 'rxjs/Subscription';
-import {MdDialogConfig} from '../dialog/dialog-config';
 import {DateAdapter} from '../core/datetime/index';
 import {createMissingDateImplError} from './datepicker-errors';
 import {ESCAPE} from '../core/keyboard/keycodes';
diff --git a/src/lib/datepicker/month-view.spec.ts b/src/lib/datepicker/month-view.spec.ts
@@ -4,13 +4,7 @@ import {By} from '@angular/platform-browser';
 import {MdMonthView} from './month-view';
 import {MdCalendarBody} from './calendar-body';
 import {MdNativeDateModule, DateAdapter, NativeDateAdapter} from '../core/datetime/index';
-
-
-// When constructing a Date, the month is zero-based. This can be confusing, since people are
-// used to seeing them one-based. So we create these aliases to make reading the tests easier.
-const JAN = 0, FEB = 1, MAR = 2, APR = 3, MAY = 4, JUN = 5, JUL = 6, AUG = 7, SEP = 8, OCT = 9,
-      NOV = 10, DEC = 11;
-
+import {JAN, MAR} from '../core/testing/month-constants';
 
 describe('MdMonthView', () => {
   beforeEach(async(() => {
diff --git a/src/lib/datepicker/year-view.spec.ts b/src/lib/datepicker/year-view.spec.ts
@@ -4,13 +4,7 @@ import {By} from '@angular/platform-browser';
 import {MdYearView} from './year-view';
 import {MdCalendarBody} from './calendar-body';
 import {MdNativeDateModule, DateAdapter, NativeDateAdapter} from '../core/datetime/index';
-
-
-// When constructing a Date, the month is zero-based. This can be confusing, since people are
-// used to seeing them one-based. So we create these aliases to make reading the tests easier.
-const JAN = 0, FEB = 1, MAR = 2, APR = 3, MAY = 4, JUN = 5, JUL = 6, AUG = 7, SEP = 8, OCT = 9,
-      NOV = 10, DEC = 11;
-
+import {FEB, JAN, MAR} from '../core/testing/month-constants';
 
 describe('MdYearView', () => {
   beforeEach(async(() => {
diff --git a/src/lib/dialog/dialog-container.ts b/src/lib/dialog/dialog-container.ts
@@ -12,7 +12,6 @@ import {
   ViewChild,
   ViewEncapsulation,
   NgZone,
-  OnDestroy,
   ElementRef,
   EventEmitter,
   Inject,
diff --git a/src/lib/dialog/dialog-injector.ts b/src/lib/dialog/dialog-injector.ts
@@ -7,8 +7,6 @@
  */
 
 import {Injector} from '@angular/core';
-import {MdDialogRef} from './dialog-ref';
-import {MdDialogContainer} from './dialog-container';
 
 /** Custom injector type specifically for instantiating components with a dialog. */
 export class DialogInjector implements Injector {
diff --git a/src/lib/dialog/dialog.spec.ts b/src/lib/dialog/dialog.spec.ts
@@ -310,7 +310,7 @@ describe('MdDialog', () => {
   });
 
   it('should allow setting the layout direction', () => {
-    let dialogRef = dialog.open(PizzaMsg, { direction: 'rtl' });
+    dialog.open(PizzaMsg, { direction: 'rtl' });
 
     viewContainerFixture.detectChanges();
 
diff --git a/src/lib/expansion/index.ts b/src/lib/expansion/index.ts
@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {NgModule, ModuleWithProviders} from '@angular/core';
+import {NgModule} from '@angular/core';
 import {CommonModule} from '@angular/common';
 import {CompatibilityModule, UNIQUE_SELECTION_DISPATCHER_PROVIDER} from '../core';
 import {
diff --git a/src/lib/paginator/index.ts b/src/lib/paginator/index.ts
@@ -9,7 +9,6 @@
 import {NgModule} from '@angular/core';
 import {CommonModule} from '@angular/common';
 import {FormsModule} from '@angular/forms';
-import {MdCommonModule, OverlayModule} from '../core';
 import {MdButtonModule} from '../button/index';
 import {MdSelectModule} from '../select/index';
 import {MdPaginator} from './paginator';
diff --git a/src/lib/paginator/paginator.spec.ts b/src/lib/paginator/paginator.spec.ts
@@ -251,15 +251,6 @@ class MdPaginatorApp {
   }
 }
 
-@Component({
-  template: `
-    <md-paginator [length]="100"></md-paginator>
-  `,
-})
-class MdPaginatorWithoutPageSizeOrOptionsApp {
-  @ViewChild(MdPaginator) mdPaginator: MdPaginator;
-}
-
 @Component({
   template: `
     <md-paginator [pageSizeOptions]="[10, 20, 30]"></md-paginator>
diff --git a/src/lib/sidenav/sidenav.spec.ts b/src/lib/sidenav/sidenav.spec.ts
@@ -1,7 +1,7 @@
 import {fakeAsync, async, tick, ComponentFixture, TestBed} from '@angular/core/testing';
 import {Component, ElementRef, ViewChild} from '@angular/core';
 import {By} from '@angular/platform-browser';
-import {MdSidenav, MdSidenavModule, MdSidenavToggleResult, MdSidenavContainer} from './index';
+import {MdSidenav, MdSidenavModule, MdSidenavContainer} from './index';
 import {A11yModule} from '../core/a11y/index';
 import {PlatformModule} from '../core/platform/index';
 import {ESCAPE} from '../core/keyboard/keycodes';
diff --git a/src/lib/snack-bar/snack-bar-container.ts b/src/lib/snack-bar/snack-bar-container.ts
@@ -26,7 +26,6 @@ import {
 import {
   BasePortalHost,
   ComponentPortal,
-  TemplatePortal,
   PortalHostDirective,
 } from '../core';
 import {MdSnackBarConfig} from './snack-bar-config';
diff --git a/src/lib/tabs/tab-nav-bar/tab-nav-bar.ts b/src/lib/tabs/tab-nav-bar/tab-nav-bar.ts
@@ -25,7 +25,6 @@ import {CanDisable, mixinDisabled} from '../../core/common-behaviors/disabled';
 import {MdRipple} from '../../core';
 import {ViewportRuler} from '../../core/overlay/position/viewport-ruler';
 import {Directionality, MD_RIPPLE_GLOBAL_OPTIONS, Platform, RippleGlobalOptions} from '../../core';
-import {Observable} from 'rxjs/Observable';
 import {Subject} from 'rxjs/Subject';
 import {takeUntil, auditTime} from '../../core/rxjs/index';
 import {of as observableOf} from 'rxjs/observable/of';
diff --git a/src/tsconfig.json b/src/tsconfig.json
diff --git a/src/universal-app/kitchen-sink/kitchen-sink.ts b/src/universal-app/kitchen-sink/kitchen-sink.ts
@@ -4,9 +4,7 @@ import {BrowserModule} from '@angular/platform-browser';
 import {
   MdAutocompleteModule,
   MdButtonModule,
-  MdButtonToggleModule,
   MdCardModule,
-  MdCheckboxModule,
   MdChipsModule,
   MdDatepickerModule,
   MdDialogModule,
diff --git a/tools/gulp/tasks/lint.ts b/tools/gulp/tasks/lint.ts
@@ -7,9 +7,7 @@ import {buildConfig} from 'material2-build-tools';
 const stylesGlob = '+(tools|src)/**/*.+(css|scss)';
 
 /** List of flags that will passed to the different TSLint tasks. */
-const tsLintBaseFlags = [
-  '-c', 'tslint.json', '+(src|e2e|tools)/**/*.ts', '--exclude', '**/node_modules/**/*'
-];
+const tsLintBaseFlags = ['-c', 'tslint.json', '--project', './tsconfig.json'];
 
 /** Path to the output of the Material package. */
 const materialOutPath = join(buildConfig.outputDir, 'packages', 'material');
diff --git a/tools/package-tools/build-bundles.ts b/tools/package-tools/build-bundles.ts
diff --git a/tools/screenshot-test/functions/util/github.ts b/tools/screenshot-test/functions/util/github.ts
diff --git a/tools/screenshot-test/src/app/viewer/viewer.component.ts b/tools/screenshot-test/src/app/viewer/viewer.component.ts
diff --git a/tsconfig.json b/tsconfig.json
diff --git a/tslint.json b/tslint.json

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`import {browser, by, element} from 'protractor';`
`2`		`-import {screenshot} from '../screenshot';`
`3`	`2`
`4`	`3`
`5`	`4`	`describe('icon', () => {`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import {Key, protractor, browser, by, element, ElementFinder} from 'protractor';`
	`1`	`+import {Key, protractor, browser, by, element} from 'protractor';`
`2`	`2`	`import {screenshot} from '../screenshot';`
`3`	`3`	`import {`
`4`	`4`	`expectToExist,`
Original file line number	Diff line number	Diff line change
`@@ -47,10 +47,8 @@ export class CdkColumnDef {`
`47`	`47`	`},`
`48`	`48`	`})`
`49`	`49`	`export class CdkHeaderCell {`
`50`		`- constructor(private columnDef: CdkColumnDef,`
`51`		`- private elementRef: ElementRef,`
`52`		`- private renderer: Renderer2) {`
`53`		- this.renderer.addClass(elementRef.nativeElement, `cdk-column-${columnDef.name}`);
	`50`	`+ constructor(columnDef: CdkColumnDef, elementRef: ElementRef, renderer: Renderer2) {`
	`51`	+ renderer.addClass(elementRef.nativeElement, `cdk-column-${columnDef.name}`);
`54`	`52`	`}`
`55`	`53`	`}`
`56`	`54`
`@@ -63,9 +61,7 @@ export class CdkHeaderCell {`
`63`	`61`	`},`
`64`	`62`	`})`
`65`	`63`	`export class CdkCell {`
`66`		`- constructor(private columnDef: CdkColumnDef,`
`67`		`- private elementRef: ElementRef,`
`68`		`- private renderer: Renderer2) {`
`69`		- this.renderer.addClass(elementRef.nativeElement, `cdk-column-${columnDef.name}`);
	`64`	`+ constructor(columnDef: CdkColumnDef, elementRef: ElementRef, renderer: Renderer2) {`
	`65`	+ renderer.addClass(elementRef.nativeElement, `cdk-column-${columnDef.name}`);
`70`	`66`	`}`
`71`	`67`	`}`