Fix linting & run linter in CI

[cgewecke]

The linter's --fix has been mangling the code here (e.g re-writing the files incorrectly).

This makes linting a very annoying manual process people sometimes skip by committing with the --no-verify flag.

The underlying problem is an older version of ganache somewhere deep in the dependency tree which corrupts the source maps. (See ganache-core 879). This can be fixed by coercing the resolved version to latest using yarn resolutions in package.json. (We don't actually use ganache in this project.)

Think this might also resolve a problem where stack traces have incorrect line numbers (have seen this occasionally)

Have also:

• linted unlinted files
• updated tslint and tslint-eslint-rules to latest
• added a lint step to the CI build job so lint failures get caught