Pre-commits are usually used to minimize busy work by the contributors, e.g., by fixing extra spacing, formatting, etc. This PR adds various basic text file checks to the repo. I also made yaml spacing a bit cleaner.
I was a bit surprised it is used for `cargo clippy` because you wouldn't want clippy's auto-fixes to be auto-applied by CI, so usually GitHub workflow simply checks runs it regularly. This is outside of the scope for this PR, but perhaps it should be removed here?
This is in response to #6715: e8bb8dffa1
The pre-commit actions have a higher MSRV than the overall project (1.70.0),
so if 1.70.0 is the configured default, this lead to problems (#6730).
This commit fixes the situation by specifying 'stable', whatever that means on the user's system.
These are not being fixed, so it does not make sense to constantly
take on new lints. The fmts may never be stabilized. Currently,
you can not even commit, because it errors out. At least switch to
lints that make it to stable when they are released.
it was showing:
[WARNING] normalizing pre-commit configuration to a top-level map. support for top level list will be removed in a future version. run: `pre-commit migrate-config` to automatically fix this.
