Route runtests.sh ruff invocations through PY_EXE

[hjmjohnson]

runtests.sh calls every other formatter (black, isort, pylint, pytype) through "${PY_EXE}" -m; the ruff invocations still called a bare ruff off PATH. is_pip_installed ruff checks $PY_EXE, so the guard and the invocation could disagree — in a clean venv built per CONTRIBUTING.md with no editable PATH entry, ./runtests.sh --codeformat fails with ruff: command not found even though ruff is installed and importable from $PY_EXE.

No behavior change when ruff happens to already be on PATH (the common case in an activated venv); this only fixes the case where it isn't.

Reproduced before/after

With $PY_EXE pointed at a venv holding ruff, and that venv absent from PATH:

$ MONAI_PY_EXE=/path/to/venv/bin/python bash -c 'PY_EXE=$MONAI_PY_EXE; ruff --version'
bash: line 1: ruff: command not found

$ MONAI_PY_EXE=/path/to/venv/bin/python bash -c 'PY_EXE=$MONAI_PY_EXE; "${PY_EXE}" -m ruff --version'
ruff 0.16.5

./runtests.sh --ruff with $PY_EXE set and the venv not on PATH now reaches All checks passed! instead of failing at the version check.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 1a50661b-109c-4c8e-820c-affd94b98651

📥 Commits

Reviewing files that changed from the base of the PR and between 303842d and 6b73bac.

📒 Files selected for processing (1)

• runtests.sh

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.

---

📝 Walkthrough

Walkthrough

runtests.sh now runs the Ruff version check and both Ruff check variants through "${PY_EXE}" -m ruff. Existing fix, unsafe-fix, and file-exclusion options remain unchanged.

Priority: ⬇️ Low — Defer this Ruff invocation fix because it is a narrow three-line test-script change with no broader product impact.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to 6b73b

Ruff now runs from the selected Python environment, avoiding PATH-dependent failures while retaining existing lint options and behavior.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description clearly explains the change, the failure scenario, and the expected behavior. It does not follow the repository template because it omits the issue reference, required section headings…	Add the required Fixes # ... line, ### Description heading, and ### Types of changes checklist. Mark the applicable test and change categories, including whether the specified integration or quick tests were run.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the main change: routing runtests.sh Ruff invocations through PY_EXE.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description clearly explains the change, the failure scenario, and the expected behavior. It does not follow the repository template because it omits the issue reference, required section headings, and the types-of-changes checklist.

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[ericspod]

Looks good thanks