Skip to content

ci: add riscv64gc-unknown-linux-gnu target#1137

Open
monosans wants to merge 4 commits intomainfrom
ci/riscv64gc-unknown-linux-gnu
Open

ci: add riscv64gc-unknown-linux-gnu target#1137
monosans wants to merge 4 commits intomainfrom
ci/riscv64gc-unknown-linux-gnu

Conversation

@monosans
Copy link
Copy Markdown
Owner

@monosans monosans commented Apr 2, 2026

No description provided.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2fba68ab-ebcb-4e2c-8757-b134ba83cd4b

📥 Commits

Reviewing files that changed from the base of the PR and between d1d2282 and db6a3c3.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Expanded build infrastructure to support Linux RISC-V 64-bit (riscv64gc) compilation targets.

Walkthrough

The CI workflow configuration was updated to add a new RISC-V 64-bit Linux GNU build target on Ubuntu 24.04 with manylinux version 2.39, extending the existing build matrix while maintaining backward compatibility with the current manylinux 2.31 configuration.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/ci.yml
Added a new RISC-V 64-bit Linux GNU (riscv64gc-unknown-linux-gnu) build matrix entry on ubuntu-24.04 targeting manylinux: "2_39" alongside the existing manylinux: "2_31" entry.
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided by the author, making it impossible to assess whether content relates to the changeset. Add a brief description explaining the purpose of adding the riscv64gc-unknown-linux-gnu target and any relevant context.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change: adding a new riscv64gc-unknown-linux-gnu build target to the CI workflow.
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/riscv64gc-unknown-linux-gnu
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch ci/riscv64gc-unknown-linux-gnu

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

@monosans monosans enabled auto-merge (squash) April 2, 2026 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant