Fix documentation to follow new directory structure

[jan-janssen]

Summary by CodeRabbit

• Documentation

  • Updated API documentation build configuration to point to the correct source directory so generated docs reflect current code.

• Tests

  • CI pipeline updated to include additional macOS test jobs as prerequisites, ensuring macOS test coverage runs before the autobot job.

[coderabbitai]

Walkthrough

Two configuration updates: the Sphinx apidoc input path in docs/source/conf.py was changed from ../../pylammpsmpi/ to ../../src/pylammpsmpi/; the GitHub Actions autobot job in .github/workflows/pipeline.yml had its needs list extended to include unittest_openmpi_mac and unittest_mpich_mac.

Changes

Cohort / File(s)	Summary
Documentation Configuration docs/source/conf.py	Updated sphinx-apidoc input path from ../../pylammpsmpi/ to ../../src/pylammpsmpi/ to point to the new source directory location.
CI Workflow .github/workflows/pipeline.yml	Extended autobot job needs list to include unittest_openmpi_mac and unittest_mpich_mac, adding macOS test jobs as prerequisites.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A rabbit peers at config lines,
Two tiny tweaks in orderly signs,
Docs now fetch from src/ with cheer,
CI waits for macOS tests near,
Hopping off to munch on code-defined vines.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The pull request title "Fix documentation to follow new directory structure" accurately reflects the primary change in the changeset: updating the sphinx-apidoc input path in docs/source/conf.py from ../../pylammpsmpi/ to ../../src/pylammpsmpi/ to align with a new source directory layout. The title is concise, specific, and clearly communicates the main objective stated in the PR. While the changeset also includes a secondary modification to .github/workflows/pipeline.yml (adding macOS test job dependencies), the title appropriately focuses on the primary documentation-related change, which aligns with the PR's stated objective.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch doc

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.41%. Comparing base (2886ce3) to head (f12442b).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@ Coverage Diff @@ ## main #392 +/- ## ======================================= Coverage 85.41% 85.41% ======================================= Files 7 7 Lines 809 809 ======================================= Hits 691 691 Misses 118 118 

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

.github/workflows/pipeline.yml (1)

205-205: Extend autobot dependencies to include macOS test variants.

The addition of unittest_openmpi_mac and unittest_mpich_mac to the autobot needs list is correct and ensures comprehensive test coverage before auto-merge. Both jobs are properly defined earlier in the workflow (lines 114 and 157).

However, this change appears tangential to the stated PR objective ("Fix documentation to follow new directory structure"). Consider whether this should be part of a separate CI workflow improvement PR.

As a minor optional refinement, consider grouping test variants by MPI implementation for clarity:

- needs: [unittest_old, unittest_openmpi, unittest_mpich, unittest_openmpi_mac, unittest_mpich_mac, pip_check, coverage] + needs: [unittest_old, unittest_openmpi, unittest_openmpi_mac, unittest_mpich, unittest_mpich_mac, pip_check, coverage]

This groups OpenMPI variants together and MPICH variants together, making the dependency structure more scannable at a glance.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 737593b and f12442b.

📒 Files selected for processing (1)

• .github/workflows/pipeline.yml (1 hunks)