Remove no-switch-case-fallthrough and no-invalid-this

[nchen63]

fixes #1861

CHANGELOG.md entry:

[removed-rule] no-switch-case-fallthrough: use tsc switch --noFallthroughCasesInSwitch instead
[removed-rule] no-invalid-this: use tsc switch --noImplicitThis instead
[recommended-removed] switch-default removed from recommended rule set

[ajafff]

I'd rather not remove those rules now. Ideally we would wait until typescript adds the ability to disable such errors: microsoft/TypeScript#11051
no-unused-variable was undeprecated for that reason, too.

Keeping those rules around doesn't add much (if any) additional maintenance burden, so I don't see a reason to rush on removing them.

In addition, typescript will eventually start using no-switch-case-fall-through in their project, see microsoft/TypeScript#14886

[adidahiya]

fair enough, I didn't realize the TS team is using no-switch-case-fall-through. @nchen63 let's keep the rules around and just remove switch-default from tslint:recommended in this PR

[adidahiya]

see above

[nchen63]

done in #2424