No problem merging as-is. I wonder what’s our standard for what to put in pre-commit vs tox, though. Like ruff format is obviously pre-commit but pylint, flake8, and ruff check not so much.

pre-commit is for really fast (subsecond?) stuff and tox is for slower stuff. This aligns with pre-commit's recommendations.

For runtimes, try: pre-commit run --all-files --verbose

I think we can lose flake8 at this point.