chore(lint): bump to tslint 4.5.0

[cexbrayat]

Update: the new rules are pushed back after 1.0 release, this PR now only contains the bump to tslint 4.5

Adds a few rules introduced by tslint 4.4 and 4.5

• arrow-return-shorthand
• no-duplicate-super
• no-import-side-effect
• no-misused-new
• no-non-null-assertion
• no-unnecessary-initializer

This introduces the new rule no-import-side-effect from @mgechev referenced in #3904.
I deliberately relaxed the rule to authorize RxJS imports and avoid tons of warnings on existing apps, but I think it's nice to have it in the tslint config, so people who want to add constraints to these imports will learn about this rule and can easily do it.

[hansl]

We need to discuss internally if we want to include the new flags for new projects, since we're in RC right now. The other option is simply to update the tslint dependency without the new flags.

[hansl]

@cexbrayat We will let the version update in, but the flags will have to go in after 1.0 is fully release. This is because it would change the way people code by adding constraints, and those are forbidden during our RC.

I'll merge this PR in when the new rules are removed. Thanks!

[cexbrayat]

@hansl Thank you for considering it. The PR now contains only the tslint bump as you requested.

If that's OK with you, I'll open a PR with the new rules after 1.0 release.

[filipesilva]

LGTM, thank you @cexbrayat.

[filipesilva]

Dismissed @hansl's review since the requested changes were made and he's away.

Changes are made, and user is away.

[filipesilva]

@cexbrayat you can open the PR now and I will mark it as post-1.0.

[cexbrayat]

@filipesilva Ok great: done in #5389 :)

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}