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...