Move test, bench, and doc requirements out of pyproject.toml

[gadomski]

Description:

I've had some discussion with folks (cc @jkeifer) that have pointed me in the direction of not including development dependencies as optional-dependencies. This PR moves test and bench out to a requirements-dev.txt, and the doc dependencies to docs/requirements.txt.

This will make it easier to:

• Keep the dev dependnecies frozen, e.g. via uv pip compile
• Avoid folks installing w/ dev dependencies when they enable all optional dependencies (e.g. with uv pip install --all-extras

PR Checklist:

• pre-commit hooks pass locally
• Tests pass (run scripts/test)
• Documentation has been updated to reflect changes, if applicable
• This PR maintains or improves overall codebase code coverage.
• Changes are added to the CHANGELOG. See the docs for information about adding to the changelog.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.20%. Comparing base (87e1a24) to head (dc09c7d).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1407   +/-   ##
=======================================
  Coverage   91.20%   91.20%           
=======================================
  Files          52       52           
  Lines        7239     7239           
  Branches     1019     1019           
=======================================
  Hits         6602     6602           
  Misses        457      457           
  Partials      180      180           

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

[jsignell]

This looks good to me! Just one typo 😝

Just wondering: are you thinking that we should move away from the scripts to rule them all model?

[gadomski]

Just wondering: are you thinking that we should move away from the scripts to rule them all model?

I personally don't love "scripts to rule them all" for libraries, I find they increase maintenance burden without much benefit. I think they're more useful for applications, when you have more resources to manage during development.

E.g. when I see scripts/test in some repo's contribution instructions, I have to go read that file (or the project's dependencies) to see what test tool we're running. But if the instructions say "run pytest", that's simpler IMO 😄.