[pre-commit.ci] pre-commit autoupdate

[pre-commit-ci]

updates:

• github.com/astral-sh/ruff-pre-commit: v0.15.21 → v0.16.5

[flamingbear]

There's actually a decent change to ruff in 0.16.x, and it's probably fine, but I want to make sure we don't break everything for folks who are just making a small change before merging this.

Update: So updating to this version will enable a bunch of rules that aren't currently enabled.

Ruff now enables 413 rules by default, up from 59 in previous versions.

It can be easily set back to the old way if people complain.

[lint]
select = ["E4", "E7", "E9", "F"]

And I verified that only the files that are touched in a PR need to be fixed with the pre-commit ci. So I recommend we go ahead and upgrade now.

[flamingbear]

I still think this is good I may have been too optimistic before, the pre-commit ci is showing lots of changes. It will need more than just an update.

pre-commit ci

[flamingbear]

@owenlittlejohns

Just FYI'ing you for a sanity check. This allows us to update ruff now, and fix problems later. The change to v0.16 was significant and I didn't want to go and update each project in one go. By adding a .ruff.toml and using the selected values, I set the tests to what was there originally as listed in the ruff migration guide.

I will probably do some test migrations for regression tests that I own before taking on anyone else's. But to migrate a single test at a time, we can push down new .ruff.toml files into sub dirs. so one in tests/hoss would set the rules for just the hoss directory.

I think this is a good plan since I didn't get this done in the IP.

[flamingbear]

I'm good with this, but I wrote most of it.