[pre-commit-ci]

updates:

• github.com/astral-sh/ruff-pre-commit: v0.15.2 → v0.16.10

[jtniehof]

This PR not only updates the pre-commit, but also runs ruff with fixes across the entire repo. Something about that is actually breaking the code, so this needs to be done after bringing everything up to snuff formatting-wise. Kicking that can, but leaving the PR open so we don't lose lock (unlikely to ever be merged as such, though).

[leowerneck]

I changed it to draft so it doesn't look like something we need to review constantly. As you said, it won't be merged in the current form anyway, so draft seems more appropriate.

[jtniehof]

Many, but not all, of the test failures here also materialized in #241 (and are fixed there).

[jtniehof]

A lot of the remaining test failures are from UP405 substitutions like:

-    sp: Optional[np.array] = None,
-    flags: Optional[np.array] = None,
+    sp: np.array | None = None,
+    flags: np.array | None = None,

leading to

TypeError: unsupported operand type(s) for |: 'builtin_function_or_method' and 'NoneType'

I think the problem is that the type is np.array, which is a factory, and the actual type is ndarray.

[leowerneck]

Is it possible to restart this PR from the latest main? I'm not sure rebasing/merging would be easy at this point, but the diff is also much larger than it needs to be.

[leowerneck]

I reset it to the latest main and bumped the version of the pre-commit hook. This should hopefully make it easier to debug.

[jtniehof]

Superseded by #246