Add more Typescript and Eslint rules

[adamseth2]

Context

Due to the increasing size of the team, code quality might suffer as everyone has their coding style. To remedy this, we should implement stricter ESLInt and ts.config rules. Due to workflow in #29 , these checks will be done automatically so this will save us a lot of time in the future.

Prereq

• Requires a deep knowledge of javascript and typescript.

Example

Explore if we should make this true (Currently it is false):
"noImplicitAny": ???

[uellenberg]

noImplicitAny should remain in use, as it helps catch issues where type inference fails. Normally when this happens (without noImplicitAny), we opt out of type checking automatically (and implicitly), which makes TypeScript less useful.

[uellenberg]

It looks like noImplicitAny is actually off. I'll look into turning it on (and fixing all the problems that it highlights).

[adamseth2]

Oops, sorry I reversed it (fixed it in the issue). Yeah ideally we want to make it on as the issue previously was people wouldn't use typescript, and as they wouldn't get any errors, they assumed they were fine. But like you said, this will require fixing a lot of stuff, but in the future will fix a lot of issues

[uellenberg]

I did some work in #35, although there's still more we can do as far as eslint goes (especially turning warnings into errors).