build(linting): execute tslint in travis

execute tslint in travis ci build
 - uses npm script until ng lint task is stable
 - lints only the typescript in the src directory
 - executes in before_script section of travis.yml to fail fast

 chore(tslint): update no-unused-variable tslint rule

 modify the no-unused-variable tslint rule to ignore
  - ddescribe
  - iit

style(button): resolves basic tslint errors

resolve the following tslint violations
  - max-line-length
  - whitespace

style(card): resolves basic tslint errors

resolve the following tslint violations
  - max-line-length

style(progress-circle): resolves basic tslint errors

resolve the following tslint violations
  - max-line-length
  - whitespace
  - eofline
  - semicolon

style(sidenav): resolves basic tslint errors

resolve the following tslint violations
  - no-inferrable-types
  - max-line-length
  - whitespace
  - eofline
  - semicolon

style(annotations): resolves basic tslint errors

resolve the following tslint violations
  - semicolon

style(demo-app): resolves basic tslint errors

resolve the following tslint violations
  - eofline

style(directives): resolves basic tslint errors

resolve the following tslint violations
  - quotemark

style(sidenav): resolves unused variable tslint errors

remove the following unused imports
  - Predicate
  - PromiseWrapper
  - TimerWrapper
remove the following unused variable
  - let openPromise: Promise<void>
ignore rule no-unused-variable on the following functions
  - closeModalSidenav_()
  - isShowingBackdrop_()
  - getMarginLeft_()
  - getMarginRight_()
  - getPositionLeft_()
  - getPositionRight_()

style(progress-circle): resolves unused variable tslint errors

remove the following unused imports
  - Attribute
  - ElementRef
  - ViewEncapsulation

style(button): resolves unused variable tslint errors

  remove the following unused imports
    - ComponentFixture
    - beforeEachProviders
    - provide
    - DebugElement
    - AsyncTestFn
  ignore rule no-unused-variable on the following function
    - testAsync()

style(one-of): resolves variable-name tslint errors

  rename OneOf to oneOf (camelCase) in following files
    - progress_circle.ts
    - sidenav.ts
    - one-of.ts
    - dir.ts

style(test_injector): resolves variable tslint errors

  remove the following unused import
    - ExceptionHandler

  ignore rule variable-name on the following variables
    - _testInjector
    - _providers
    - _injector
    - _instantiated

style(main): resolves variable tslint errors

  remove the following unused imports
    - HashLocationStrategy
    - LocationStrategy
    - provide

build(linting): update tslint version

Update to 3.5.0 to make user of ignore patterns

style(sidenav): remove individual tslint rule ignore statements

  - remove all individual ignore statements for no-unused-variable on private class members
  - add regex to no-unused-variable ignore-pattern

style(sidenav.spec): modify line break

  - break after toEqual to improve readability

style(test_injector): resolves variable tslint errors

  rename var to let

  remove all individual ignore statements for variable-name

  move _ to end of private var name

  rename testInjector to testInjectorCache
    - resolves no-shadowed-variable error in setBaseTestProviders()
    - resolves no-shadowed-variable error in resetBaseTestProviders()

style(button.spec): remove unused testAsync function

  - remove unused testAsync function
  - remove now unused module imports

style(karma-test-shim): increase jasmine default timeout

  increase jasmine.DEFAULT_TIMEOUT_INTERVAL from 2000 to 4000ms.
    - Temporary resolution of Async callback timeouts in tests

style(button): fix whitespace

resolve whitespace issue in setClassList()

style(one-of): modified case of OneOf

  - Fixed case for OneOf dec. factory
  - Modified export in one-of.ts

Closes #134
This commit is contained in:
d3viant0ne
2016-03-07 18:47:34 -06:00
committed by Jeremy Elbourn
parent 10c9aed401
commit 425db8aef5
18 changed files with 98 additions and 109 deletions
+2 -1
View File
@@ -12,6 +12,7 @@
"dartanalyzer": "cd dist/dart && pub install && cd ../.. && ts-node scripts/ci/dart_analyzer",
"demo-app": "cd src && ng serve",
"test": "karma start test/karma.conf.js",
"tslint" : "tslint -c tslint.json 'src/**/*.ts'",
"typings": "typings install --ambient"
},
"version": "2.0.0-alpha.0",
@@ -53,7 +54,7 @@
"symlink-or-copy": "^1.0.1",
"ts-node": "^0.5.5",
"ts2dart": "^0.7.27",
"tslint": "^3.2.2",
"tslint": "^3.5.0",
"typescript": "^1.7.5",
"typings": "^0.6.8",
"which": "^1.2.4"