Remove eslint typescript warnings
The following discussion from !1766 (merged) should be addressed:
-
@georgkrause started a discussion: (+2 comments). Relevant part:
no-undef
provided by eslint does not know about some types and throws errors. tslint handles this rule correctly on its own.
no-this-alias
throws on everyconst self = this
. It's everywhere in the code. Dunno who to blame but when you use arrow functions, there's no need to alias this. Arrow functions do not have their ownthis
, they usethis
from their parent. When I finish rewriting, it shouldn't be a problem anymore and we can remove this rule.
no-empty-function
- there were a couple of empty functions, I just wanted to pass the linting tests. When I rewrite the logic to typescript, we also can remove this line.