Repository navigation
test(mpi): ♻️ remove unmaintained pytest-mpi - #415
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Assisted-by: GitHub Copilot:default Co-authored-by: robertodr <3708689+robertodr@users.noreply.github.com>
|
Docs preview: https://pr-415.monoprop-docs.pages.dev |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #415 +/- ##
=======================================
Coverage 97.99% 97.99%
=======================================
Files 14 14
Lines 797 797
Branches 111 111
=======================================
Hits 781 781
Misses 11 11
Partials 5 5
Flags with carried forward coverage won't be shown. Click here to find out more. |
|



🤖 AI text below 🤖
Summary
Remove the unmaintained
pytest-mpidependency while preserving the existing MPI workflow. Use standard pytest hooks instead ofpytest-isolate-mpi, whose unconditional MPI-launcher requirement conflicts with serial-only test environments.Changes
--with-mpi, minimum-rank markers, VS Code activation, and rank-zero reporting. Existing MPI and coverage recipes remain unchanged.Checklist
AI/LLM disclosure