Don't auto-fix when linting client

[gabemart]

Currently we have npm run lint set to autofix where possible

I think this is leading to slight weirdness - if you merge something that is auto-fixable without actually running lint locally, it will pass CI (because the CI will auto-fix when it runs the lint step) but the actual code as-merged is not linted

I suggest we go back to the old system of npm run lint simply running the linter and returning a pass/fail, and npm run lint:fix running the linter with the --fix flag