Remove npm audit warnings and tslint deprecated rule warning

[twindual]

What:

This update fixes:
-npm audit warnings
-tslint deprecated rule warning
-adds a new contributor
-adds Google firestore package (to use auth package)

Issue number: N/A

[codecov-io]

Codecov Report

Merging #485 into master will decrease coverage by 0.18%.
The diff coverage is n/a.

@@ Coverage Diff @@ ## master #485 +/- ## ========================================== - Coverage 72.1% 71.92% -0.19%  ========================================== Files 65 65 Lines 552 552 Branches 34 34 ========================================== - Hits 398 397 -1  - Misses 144 145 +1  Partials 10 10

Impacted Files	Coverage Δ
...tarter/src/app/core/animations/route.animations.ts	80% <0%> (-10%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2c8e90...61fea3b. Read the comment docs.

[twindual]

Wondering if the codecov/project test failed because of different dependencies, as there were no code changes there?

Any ideas?

[tomastrajan]

Hi @twindual!

Thanks for the contribution, please see the review comments.

[tomastrajan]

Why did this file have to change?

[tomastrajan]

Builders / CLI sometimes breaks with minors, would keep it to parch version ~

[tomastrajan]

Don't see any reason to add dependencies while not using them, also firebase is currently out of scope of this project, it is meant to be beginners friendly and is already pretty complex...

Also store freeze is not needed with ngrx 8 so please remove all the extra added dependencies...