Move to pre-commit only

[mikealfare]

Problem

Our linters and typechecking tools create a lot of dependabot PRs. This is aggravated by the fact that we keep one set of reqs in dev-requirements.txt and another in .pre-commit-config.yaml.

Solution

• move all reqs and config for linters and typecheckers into .pre-commit-config.yaml
• run pre-commit autoupdate
• run pre-commit run --all-files
• fix resulting error from running pre-commit
• remove make recipes that are no longer relative or are duplicative

Out of scope:

• as a follow up, create a weekly action that runs pre-commit autoupdate against a temp branch, runs linting and typechecking, and merges on success or creates a PR on failure

Checklist

• I have read the contributing guide and understand what's expected of me
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

[github-actions]

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the dbt-snowflake contributing guide.