Fix ruff

[jan-janssen]

Summary by CodeRabbit

• Chores
  • Updated pre-commit configuration to refine the scope of automated linting checks.

[coderabbitai]

Walkthrough

Updated the pre-commit configuration file to change the ruff lint filter path from ^executorlib/ to ^src/executorlib/, restricting the linting hook to operate only on files within the new source directory structure.

Changes

Cohort / File(s)	Summary
Pre-commit Configuration .pre-commit-config.yaml	Updated ruff lint filter path from ^executorlib/ to ^src/executorlib/ to scope linting to the new source directory structure

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A whisper on the config winds,
A path adjusted, neat and tidy,
From old to new, the linter spins,
Scoped to src/ on Friday!
✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title Check	❓ Inconclusive	The title "Fix ruff" is related to the changeset, as the pull request does address a ruff configuration issue in the .pre-commit-config.yaml file. However, the title is vague and generic, using the non-descriptive term "Fix" without specifying what aspect of ruff is being fixed or the nature of the change. A teammate scanning the git history would not understand that this specifically updates the ruff lint filter path from ^executorlib/ to ^src/executorlib/, making it difficult to grasp the purpose of this commit from the title alone.	Consider revising the title to be more descriptive and specific about the change, such as "Update ruff pre-commit filter path from executorlib to src/executorlib" or "Fix ruff lint filter path in pre-commit config". This would clearly communicate what is being fixed and why, making the commit history more informative for future reference.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
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 Fix

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9a428dc and 26272a6.

📒 Files selected for processing (1)

• .pre-commit-config.yaml (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (19)

• GitHub Check: unittest_openmpi (ubuntu-latest, 3.12)
• GitHub Check: unittest_openmpi (macos-latest, 3.13)
• GitHub Check: unittest_openmpi (ubuntu-latest, 3.11)
• GitHub Check: unittest_openmpi (ubuntu-latest, 3.13)
• GitHub Check: unittest_openmpi (ubuntu-24.04-arm, 3.13)
• GitHub Check: unittest_flux_openmpi
• GitHub Check: unittest_openmpi (ubuntu-22.04-arm, 3.13)
• GitHub Check: unittest_flux_mpich
• GitHub Check: notebooks
• GitHub Check: unittest_slurm_mpich
• GitHub Check: unittest_mpich (ubuntu-latest, 3.11)
• GitHub Check: notebooks_integration
• GitHub Check: unittest_mpich (ubuntu-latest, 3.13)
• GitHub Check: unittest_mpich (ubuntu-24.04-arm, 3.13)
• GitHub Check: unittest_mpich (ubuntu-22.04-arm, 3.13)
• GitHub Check: benchmark (ubuntu-latest, 3.13, .ci_support/environment-mpich.yml)
• GitHub Check: benchmark (ubuntu-latest, 3.13, .ci_support/environment-openmpi.yml)
• GitHub Check: unittest_old
• GitHub Check: unittest_win

🔇 Additional comments (1)

.pre-commit-config.yaml (1)

8-8: The path update is correct; ruff-format asymmetry is intentional.

The change properly updates the ruff-lint hook's file scope from ^executorlib/ to ^src/executorlib/, matching the actual directory structure. The ruff-format hook intentionally lacks a files: directive (and never had one in recent history), allowing it to format all Python files including tests, while linting is scoped to source code only. This asymmetry is by design, not an oversight.

---

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 93.24%. Comparing base (9a428dc) to head (26272a6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #847   +/-   ##
=======================================
  Coverage   93.24%   93.24%           
=======================================
  Files          38       38           
  Lines        1791     1791           
=======================================
  Hits         1670     1670           
  Misses        121      121           

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