Skip to content
This repository has been archived by the owner on Aug 28, 2020. It is now read-only.

Commit

Permalink
remove problematic rules
Browse files Browse the repository at this point in the history
@brandyscarney says:

> here are the ones I ran into issues with in the framework:
>
> * `no-unnecessary-type-assertion` is throwing errors when we cast as an HTMLElement - which is necessary because otherwise it queries as an `Element` and throws errors that the properties don’t exist, this is a bug in TS: palantir/tslint#3540
> * `prefer-for-of` is throwing errors on node lists which is also a bug: palantir/tslint#2927
> * `no-conditional-assignment` is used in DOM controller
  • Loading branch information
imhoffd committed Jan 18, 2018
1 parent c2e472c commit 065e3e9
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions tslint.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ module.exports = {
"jsdoc-format": [true, "check-multiline-start"],
"linebreak-style": [true, "LF"],
"new-parens": true,
"no-conditional-assignment": true,
"no-debugger": true,
"no-duplicate-super": true,
"no-duplicate-switch-case": true,
Expand All @@ -32,7 +31,6 @@ module.exports = {
"no-string-throw": true,
"no-switch-case-fall-through": true,
"no-trailing-whitespace": true,
"no-unnecessary-type-assertion": true,
"no-unsafe-finally": true,
"no-unused-variable": true,
"no-var-keyword": true,
Expand All @@ -53,7 +51,6 @@ module.exports = {
},
"prefer-conditional-expression": true,
"prefer-const": true,
"prefer-for-of": true,
"quotemark": {"options": ["single"]},
"radix": true,
"semicolon": {"options": ["always"]},
Expand Down

0 comments on commit 065e3e9

Please sign in to comment.