Skip to content

Commit

Permalink
chore: Remove unused imports and enable checks for unused imports as …
Browse files Browse the repository at this point in the history
…part of build
  • Loading branch information
kevmoo authored and mhevery committed Feb 19, 2015
1 parent 821f008 commit fb5b168
Show file tree
Hide file tree
Showing 12 changed files with 8 additions and 17 deletions.
2 changes: 0 additions & 2 deletions modules/angular2/src/core/life_cycle/life_cycle.js
@@ -1,8 +1,6 @@
import {FIELD, print} from 'angular2/src/facade/lang';
import {ChangeDetector} from 'angular2/change_detection';
import {VmTurnZone} from 'angular2/src/core/zone/vm_turn_zone';
import {ExceptionHandler} from 'angular2/src/core/exception_handler';
import {ListWrapper} from 'angular2/src/facade/collection';
import {isPresent} from 'angular2/src/facade/lang';

export class LifeCycle {
Expand Down
Expand Up @@ -5,7 +5,6 @@ import {List, ListWrapper, MapWrapper, StringMapWrapper} from 'angular2/src/faca

import {Parser} from 'angular2/src/change_detection/parser/parser';
import {Lexer} from 'angular2/src/change_detection/parser/lexer';
import {arrayChangesAsString, kvChangesAsString} from './util';

import {ChangeDispatcher, DynamicChangeDetector, ChangeDetectionError, ContextWithVariableBindings,
PipeRegistry, NO_CHANGE,
Expand Down
@@ -1,6 +1,5 @@
import {describe, beforeEach, it, expect, iit, ddescribe, el} from 'angular2/test_lib';
import {isPresent, isBlank} from 'angular2/src/facade/lang';
import {DOM} from 'angular2/src/facade/dom';
import {List, ListWrapper, MapWrapper} from 'angular2/src/facade/collection';

import {ProtoElementInjectorBuilder} from 'angular2/src/core/compiler/pipeline/proto_element_injector_builder';
Expand Down
Expand Up @@ -8,7 +8,6 @@ import {CompileElement} from 'angular2/src/core/compiler/pipeline/compile_elemen
import {CompileStep} from 'angular2/src/core/compiler/pipeline/compile_step'
import {CompileControl} from 'angular2/src/core/compiler/pipeline/compile_control';
import {NativeShadowDomStrategy} from 'angular2/src/core/compiler/shadow_dom_strategy';
import {DOM} from 'angular2/src/facade/dom';
import {MapWrapper} from 'angular2/src/facade/collection';

export function main() {
Expand Down
@@ -1,7 +1,6 @@
import {describe, beforeEach, expect, it, iit, ddescribe, el} from 'angular2/test_lib';
import {TextInterpolationParser} from 'angular2/src/core/compiler/pipeline/text_interpolation_parser';
import {CompilePipeline} from 'angular2/src/core/compiler/pipeline/compile_pipeline';
import {DOM} from 'angular2/src/facade/dom';
import {MapWrapper} from 'angular2/src/facade/collection';
import {Lexer, Parser} from 'angular2/change_detection';
import {CompileElement} from 'angular2/src/core/compiler/pipeline/compile_element';
Expand Down
@@ -1,5 +1,4 @@
import {describe, beforeEach, it, expect, iit, ddescribe, el} from 'angular2/test_lib';
import {isPresent} from 'angular2/src/facade/lang';
import {MapWrapper} from 'angular2/src/facade/collection';

import {ViewSplitter} from 'angular2/src/core/compiler/pipeline/view_splitter';
Expand Down
1 change: 0 additions & 1 deletion modules/angular2/test/core/compiler/selector_spec.js
Expand Up @@ -2,7 +2,6 @@ import {describe, it, expect, beforeEach, ddescribe, iit, xit, el} from 'angular
import {SelectorMatcher} from 'angular2/src/core/compiler/selector';
import {CssSelector} from 'angular2/src/core/compiler/selector';
import {List, ListWrapper, MapWrapper} from 'angular2/src/facade/collection';
import {DOM} from 'angular2/src/facade/dom';

export function main() {
describe('SelectorMatcher', () => {
Expand Down
@@ -1,6 +1,5 @@
import {describe, xit, it, expect, beforeEach, ddescribe, iit, el} from 'angular2/test_lib';

import {DOM} from 'angular2/src/facade/dom';
import {StringMapWrapper, MapWrapper, List} from 'angular2/src/facade/collection';
import {isPresent, Type} from 'angular2/src/facade/lang';

Expand Down
Expand Up @@ -5,7 +5,6 @@ import {TemplateLoader} from 'angular2/src/core/compiler/template_loader';
import {Template} from 'angular2/src/core/annotations/template';

import {PromiseWrapper} from 'angular2/src/facade/async';
import {Type, stringify, isPresent} from 'angular2/src/facade/lang';

import {XHRMock} from 'angular2/src/mock/xhr_mock';

Expand Down
1 change: 0 additions & 1 deletion modules/angular2/test/mock/xhr_mock_spec.js
Expand Up @@ -2,7 +2,6 @@ import {describe, beforeEach, it, expect, ddescribe, iit, el, IS_DARTIUM} from '
import {XHRMock} from 'angular2/src/mock/xhr_mock';
import {PromiseWrapper, Promise} from 'angular2/src/facade/async';
import {isPresent} from 'angular2/src/facade/lang';
import {ListWrapper} from 'angular2/src/facade/collection';

export function main() {
describe('XHRMock', () => {
Expand Down
1 change: 0 additions & 1 deletion modules/benchpress/src/sample_description.js
@@ -1,6 +1,5 @@
import { StringMapWrapper, ListWrapper, StringMap } from 'angular2/src/facade/collection';
import { bind, OpaqueToken } from 'angular2/di';
import { Sampler } from './sampler';
import { Validator } from './validator';
import { Metric } from './metric';
import { Options } from './sample_options';
Expand Down
13 changes: 8 additions & 5 deletions tools/build/dartanalyzer.js
Expand Up @@ -54,11 +54,14 @@ module.exports = function(gulp, plugins, config) {
return;
}
if (line.match(/Unused import/)) {
return;
}
if (line.match(/\[info\]/)) {
// Ignore [info] warnings
return;
if (line.match(/_analyzer\.dart/)) {
return;
}

//TODO: remove this work-around once #704 is fixed
if (line.match(/\/test\/core\/compiler\/view_.*spec\.dart/)) {
return;
}
}
if (line.match(/\[hint\]/)) {
hintCount++;
Expand Down

0 comments on commit fb5b168

Please sign in to comment.