ci: consolidate pyproject2conda invocation through the prek hook

[drbenvincent]

Summary

Replaces the bespoke pip install pyproject2conda + pyproject2conda yaml ... + tail -n +10 ... | diff -u sequence in the check-environment-yml job with a single call to the existing pyproject2conda-yaml prek hook, then a git diff --exit-code against the committed environment.yml.

After this PR:

• The pin for pyproject2conda lives only in .pre-commit-config.yaml (the prek hook rev). Bumping the tool requires editing exactly one file.
• The argument list lives only in .pre-commit-config.yaml. Local prek run --all-files and the check-environment-yml CI job exercise the same hook with the same args by construction.
• The 9-line tail -n +10 magic-number header skip is gone; the prek hook writes the workspace environment.yml in place, so a plain git diff is sufficient.

Why

#878 unblocked CI by pinning pyproject2conda<0.23 in the workflow, but it had to add an inline comment reminding maintainers to bump the workflow pin and the prek hook rev in lockstep. That's exactly the dual-pin smell that motivates this consolidation. See #879 for the design discussion.

Notes for reviewers

• Uses the same j178/prek-action@cbc2f23... SHA pin already used by the prek job in this workflow, so no new third-party action is introduced.
• permissions: contents: read is unchanged; the job still does not need write access.
• The new error message tells contributors exactly what to run locally to fix a sync failure (prek run pyproject2conda-yaml --all-files), which the previous diff -u did not.

Test plan

• check-environment-yml job passes on this PR (no drift between environment.yml and pyproject.toml).
• Verify the job actually fails when environment.yml is out of date (e.g. push a deliberate desync to a scratch branch and confirm the new error message appears).
• Confirm the prek hook's pinned rev: v0.22.1 is the version actually executed in CI (visible in the action logs).

Related

• Builds on ci: pin pyproject2conda==0.22.1 to work around 0.23.0 regression #878 (the immediate unblock).
• Closes Consolidate CI pyproject2conda invocation through the prek hook #879 (consolidate via prek hook).
• Partially mitigates Fragile header-skip in check-environment-yml diff comparison #880 (header-skip fragility) for this specific job; Fragile header-skip in check-environment-yml diff comparison #880 stays open as the general principle.

Made with Cursor

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.14%. Comparing base (7b306e6) to head (a594ca3).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #882   +/-   ##
=======================================
  Coverage   95.14%   95.14%           
=======================================
  Files          92       92           
  Lines       14860    14860           
  Branches      890      890           
=======================================
  Hits        14138    14138           
  Misses        505      505           
  Partials      217      217           

☔ 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.

[read-the-docs-community]

Documentation build overview

📚 causalpy | 🛠️ Build #32484998 | 📁 Comparing 00c8f16 against latest (deb8774)

  🔍 Preview build  

1 file changed

± 404.html