Ci/add riscv runner

[ChanTsune]

Summary by CodeRabbit

• Chores
  • Added lint configuration to recognize a self-hosted runner label for validation.
  • Expanded workflow triggers to monitor additional CI/configuration files so updates to tooling configs run linters.
  • Expanded the CI test matrix with a new Ubuntu RISC‑V runner and conditional job timeout for RISC‑V jobs.
  • Updated CI setup to install extra build tooling and made feature-exclusion configurable per runner type.

[coderabbitai]

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

Adds an actionlint config declaring a ubuntu-24.04-riscv self-hosted-runner label, extends actions-lint workflow triggers to include .github/actionlint.yaml and .github/zizmor.yml, and updates CI (.github/workflows/test.yml) to add a RISC‑V matrix entry, conditional timeout, and install cmake with adjusted exclude features.

Changes

Cohort / File(s)	Summary
Actionlint config /.github/actionlint.yaml	New file: declares self-hosted-runner.labels: [ubuntu-24.04-riscv] for actionlint validation.
Lint workflow triggers /.github/workflows/actions-lint.yml	Extend pull_request.paths and push.paths to also trigger on .github/actionlint.yaml and .github/zizmor.yml.
CI test workflow /.github/workflows/test.yml	Add ubuntu-24.04-riscv to Tier 1 matrix; make timeout-minutes conditional (120 when matrix.os contains riscv, else 60); install cmake alongside libacl1-dev on Ubuntu/MSRV jobs; set EXCLUDE_FEATURES to wasm,zlib-ng for RISC‑V and wasm otherwise.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• ✨ Add riscv64gc-unknown-linux-gnu release target #2893 — Adds/configures the ubuntu-24.04-riscv self-hosted runner and related CI matrix changes.
• 👷 Add actionlint and zizmor workflow for GitHub Actions linting #2840 — Introduces actionlint/workflow linting changes related to adding an actionlint config.
• Ci/vm path filter #2745 — Modifies GitHub Actions workflow configurations, including test.yml triggers and matrix adjustments.

Poem

🐰 I hopped into CI with a jaunty spin,
A tiny RISC‑V label tucked within,
Actionlint checks my tiny hat, hooray—
CMake joins the dance and tests make way,
Build carrots crisp, and bunnies bound to play.

🚥 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 'Ci/add riscv runner' directly and clearly summarizes the main change: adding a RISC-V runner to CI configuration across workflow and configuration files.
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
• Commit unit tests in branch ci/add-riscv-runner

---

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

[gemini-code-assist]

Code Review

This pull request updates the libz-ng-sys dependency to version 1.1.25 and adds a new RISC-V runner label to the actionlint configuration. Feedback indicates that the implementation is currently incomplete because the GitHub Actions workflow files have not been updated to actually utilize the new runner.

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

.github/workflows/test.yml (1)

95-95: Consider adding a comment explaining why zlib-ng is excluded on RISC-V.

The exclusion is likely due to architecture-specific compilation issues with zlib-ng on RISC-V. A brief comment would help future maintainers understand this decision.

📝 Suggested documentation

        env:
          RUST_BACKTRACE: 1
+         # zlib-ng excluded on RISC-V due to compilation/architecture support issues
          EXCLUDE_FEATURES: ${{ contains(matrix.os, 'riscv') && 'wasm,zlib-ng' || 'wasm' }}

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In @.github/workflows/test.yml at line 95, Add a short inline comment above the
EXCLUDE_FEATURES line in .github/workflows/test.yml explaining why zlib-ng is
excluded on RISC-V (e.g., known architecture-specific build/compatibility issues
or a referenced bug/PR), so maintainers know the rationale for the conditional
EXCLUDE_FEATURES: ${{ contains(matrix.os, 'riscv') && 'wasm,zlib-ng' || 'wasm'
}}; if there is a specific issue/PR or a brief error message, include that
reference in the comment.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/test.yml:
- Around line 90-95: The GitHub Actions step named "run test" uses bash-style
env expansion for EXCLUDE_FEATURES which fails on Windows pwsh; update the step
that runs `cargo hack test ... --exclude-features "$EXCLUDE_FEATURES"` to
explicitly set shell: bash so the `$EXCLUDE_FEATURES` expansion works
cross-platform (keep the existing env entries RUST_BACKTRACE and
EXCLUDE_FEATURES unchanged); ensure the `shell: bash` line is added at the same
level as `run:` in the "run test" step.

---

Nitpick comments:
In @.github/workflows/test.yml:
- Line 95: Add a short inline comment above the EXCLUDE_FEATURES line in
.github/workflows/test.yml explaining why zlib-ng is excluded on RISC-V (e.g.,
known architecture-specific build/compatibility issues or a referenced bug/PR),
so maintainers know the rationale for the conditional EXCLUDE_FEATURES: ${{
contains(matrix.os, 'riscv') && 'wasm,zlib-ng' || 'wasm' }}; if there is a
specific issue/PR or a brief error message, include that reference in the
comment.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c3ca4371-82b7-4fb5-b682-4ea83a5e5653

📥 Commits

Reviewing files that changed from the base of the PR and between 7d2fe2f and 32a14fc.

📒 Files selected for processing (1)

• .github/workflows/test.yml