feat(chore): configure pre-commit hook

[jamesgeorge007]

pre-commit hooks come in handy in the entire Dev workflow. Now the lint script is automatically executed before making a commit on staged files.

[ghost]

This may be fine on your own fork, but I'm not sure this is appropriate for the project as a whole.

I, for one, create many commits and tend to rebase/re-arrange/etc and these pre-commit hooks will add an unbearable amount of overhead to that workflow.

[Conduitry]

Thanks, but yeah, this is something that we don't want to force on everyone. I'm not thrilled about introducing all of these extra dependencies, or about introducing more stuff that gets run on postinstall, or about automatically adding git hooks for people that they don't necessarily want.

We already have CI to catch this stuff once a PR is opened. And I often commit partially complete code while working on something, and I don't want to have to wait for linting on every commit or have the commit fail because I haven't tidied up all of the linting errors in my intermediate code.