Ruff as part of PR auto-format by ShaharNaveh · Pull Request #6421 · RustPython/RustPython
Walkthrough
Two GitHub Actions workflow files are modified to enhance Ruff linter integration. The CI workflow adds a version display flag to the Ruff action, while the PR auto-commit workflow introduces new Ruff format and import check steps into the code formatting pipeline.
Changes
| Cohort / File(s) | Change Summary |
|---|---|
GitHub Actions Workflows - Ruff Tooling .github/workflows/ci.yaml, .github/workflows/pr-auto-commit.yaml |
CI workflow: adds --version argument to ruff-action step for explicit version display. PR auto-commit workflow: introduces Ruff setup via astral-sh/ruff-action (v0.14.9), followed by ruff format and ruff check --select I --fix steps in the code formatting sequence. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~8 minutes
- Both changes are straightforward workflow configuration additions with no complex logic or cross-file coordination.
- The
pr-auto-commit.yamladditions introduce sequential Ruff steps that follow a clear pattern and should be verified for correct placement relative to existingcargo fmtexecution.
Possibly related PRs
- Update ruff CI version #6418 — Modifies GitHub Actions workflows to adjust astral-sh/ruff-action usage and Ruff invocation patterns with version specification.
- Configure dependabot to ignore ruff updates #6185 — Configures Dependabot to ignore Ruff updates, complementing the Ruff tooling changes introduced in this PR.
Suggested reviewers
- youknowone
Poem
🐰 A rabbit hops through the CI,
With Ruff now checking every line so nigh,
Version flags shine, imports align,
Auto-commit workflows now refine! ✨
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 title accurately describes the main objective: integrating Ruff into the PR auto-format workflow across both ci.yaml and pr-auto-commit.yaml files. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✨ Finishing touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
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.
Comment @coderabbitai help to get the list of available commands and usage tips.