Add actionlint step by ShaharNaveh · Pull Request #7464 · RustPython/RustPython

ShaharNaveh

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

@coderabbitai resume to resume automatic reviews.
@coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

▶️ Resume reviews
🔍 Trigger review

📝 Walkthrough

Walkthrough

Inserted a pinned reviewdog/action-actionlint step into the format_check job of .github/workflows/pr-format.yaml, and made several shell/quoting and minor script formatting edits in .github/workflows/ci.yaml. No other workflow control flow or step ordering was changed.

Changes

Cohort / File(s)	Summary
Format workflow `.github/workflows/pr-format.yaml`	Added a `reviewdog/action-actionlint` step to the `format_check` job, pinned to commit `0d952c597ef8459f634d7145b0b044a9699e5e43` (comment `v1.71.0`). Inserted immediately after `actions/checkout`; no other steps' conditions, inputs, commands, or ordering were modified.
CI workflow — shell/script tweaks `.github/workflows/ci.yaml`	Converted the `install prettier` step from a single-line `run` into a multi-line script. Quoted `export PATH` in the `wasm` job (`export PATH="$PATH:$(pwd)/../../geckodriver"`). Replaced backticked `pwd` with `$(pwd)` and quoted `--dir` and script path arguments in `wasm-wasi` `wasmer run` invocations to avoid shell word-splitting. No functional changes beyond quoting and script formatting.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

youknowone

Poem

🐰 I hopped through lines of YAML bright,
Pinned a lint step, snug and tight.
Quoted paths and fixed the run,
Tiny changes, now we're done —
I munch a carrot in delight. 🥕

🚥 Pre-merge checks | ✅ 3

✅ 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 'Add actionlint step' directly describes the main change: adding a new actionlint action to the workflow. However, the pull request also modifies ci.yaml with additional workflow improvements (Prettier installation refactoring, PATH quoting, and pwd command improvements), which are not captured in the title.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

📝 Coding Plan

Generate coding plan for human review comments

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

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