https://github.blog/changelog/2022-05-11-default-to-pr-titles-for-squash-merge-commit-messages/

I have turned this option on - so now single commit PRs will use the PR title isntead of the commit message.
Now that means there's no reason for us to have commitlint installed or setup as a husky hook.
It's been a pet-peeve for a while now that local commits must follow the semantic commit standard - and I often bypass validations because of it!

We should remove the commit linter entirely.

Relatedly - it looks like we lost our CI job which validated the PR title - we should have validation setup for it there so that we can ensure that PR titles match the required format.