Skip to content

test(mpi): ♻️ remove unmaintained pytest-mpi - #415

Merged
robertodr merged 5 commits into
mainfrom
copilot/remove-pytest-mpi
Oct 23, 2026
Merged

robertodr merged 5 commits into
mainfrom
copilot/remove-pytest-mpi

Conversation

Copilot AI commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

🤖 AI text below 🤖

Summary

Remove the unmaintained pytest-mpi dependency while preserving the existing MPI workflow. Use standard pytest hooks instead of pytest-isolate-mpi, whose unconditional MPI-launcher requirement conflicts with serial-only test environments.

Changes

  • Dependencies: Remove the plugin dependency and requirement; refresh both lockfiles.
  • Compatibility: Preserve --with-mpi, minimum-rank markers, VS Code activation, and rank-zero reporting. Existing MPI and coverage recipes remain unchanged.

Checklist

  • Add 15 focused regression cases covering selection, rank requirements, and reporting.
  • Document the plugin-free MPI setup.

AI/LLM disclosure

  • GitHub Copilot helped generate the implementation, tests, and this description.

@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b0b71b75-efd4-474a-90ae-66e17b5e6da9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

Assisted-by: GitHub Copilot:default

Co-authored-by: robertodr <3708689+robertodr@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove pytest-mpi which is no longer actively maintained test(mpi): ♻️ remove unmaintained pytest-mpi Oct 19, 2026
Copilot AI requested a review from robertodr October 19, 2026 13:09
@robertodr
robertodr marked this pull request as ready for review October 19, 2026 13:13
@github-actions github-actions Bot added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file python labels Oct 19, 2026
@github-actions

Copy link
Copy Markdown

Docs preview: https://pr-415.monoprop-docs.pages.dev

@codecov

codecov Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.99%. Comparing base (794f239) to head (deb1865).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

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           
Flag Coverage Δ
cpp 97.99% <ø> (ø)

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

@robertodr
robertodr enabled auto-merge (squash) October 22, 2026 08:26
@robertodr
robertodr merged commit 115980d into main Oct 23, 2026
42 checks passed
@robertodr
robertodr deleted the copilot/remove-pytest-mpi branch October 23, 2026 09:32
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove pytest-mpi which is no longer actively maintained

4 participants