[pre-commit.ci] pre-commit autoupdate

[pre-commit-ci]

updates:

• github.com/pre-commit/mirrors-mypy: v2.1.0 → v2.3.0
• github.com/astral-sh/ruff-pre-commit: v0.15.20 → v0.16.2

[sourcery-ai]

Hey - I've reviewed your changes and they look great!

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.62%. Comparing base (a1367c0) to head (06f608a).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #360      +/-   ##
==========================================
- Coverage   94.65%   94.62%   -0.03%     
==========================================
  Files          77       77              
  Lines        3815     3815              
  Branches      248      248              
==========================================
- Hits         3611     3610       -1     
  Misses        137      137              
- Partials       67       68       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[omosola]

Overall, looks good to me. 👍

I added comments related to the positional argument changes. Those were the main things I would update within this PR.

[PGijsbers]

Decided to make all arguments keyword-only instead of changing the call sites. While it's one of the more interpretable arguments (which is why it was kept as optionally positional), it's also one of the less interesting ones so I don't really want to move that up the parameter order.

[omosola]

Decided to make all arguments keyword-only instead of changing the call sites. While it's one of the more interpretable arguments (which is why it was kept as optionally positional), it's also one of the less interesting ones so I don't really want to move that up the parameter order.

Works for me 👍