Reqs in pyproject

[benjeffery]

I've been thinking about tidying up the requirements files for some time. The modern way it to move them into pyproject.toml. Do people like this?

[molpopgen]

Yes, but there are some caveats. Some tools that work with pyproject may not respect pinning build dependencies. Example

But overall, switching to pyproject for my projects has been a big win, especially when using uv to manage everything.

[jeromekelleher]

I think it's a win too. How does this gel with doing things with conda though?

[molpopgen]

conda makes things harder, of course. AFAIK, it doesn't understand pyproject.toml. The language-agnosticism of conda seems to imply that its requirements files are a separate thing.

[benjeffery]

Yeah, we'd need to retain a conda text reqs for gsl on windows/mac, or hardcode it in CI.

[jeromekelleher]

That's not so bad.

[jeromekelleher]

I'm going to mute this one @benjeffery - ping me if you want input

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.23%. Comparing base (7a0ff9d) to head (87b26a6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2413   +/-   ##
=======================================
  Coverage   91.23%   91.23%           
=======================================
  Files          20       20           
  Lines       11831    11831           
  Branches     2296     2296           
=======================================
  Hits        10794    10794           
  Misses        568      568           
  Partials      469      469           

Flag	Coverage Δ
C	91.23% <ø> (ø)
python	98.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

[benjeffery]

@jeromekelleher Ready for review