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
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] 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
Problem
Our linters and typechecking tools create a lot of
dependabotPRs. This is aggravated by the fact that we keep one set of reqs indev-requirements.txtand another in.pre-commit-config.yaml.Solution
.pre-commit-config.yamlpre-commit autoupdatepre-commit run --all-filespre-commitmakerecipes that are no longer relative or are duplicativeOut of scope:
pre-commit autoupdateagainst a temp branch, runs linting and typechecking, and merges on success or creates a PR on failureChecklist