Skip to content

ci: add loongarch64-unknown-linux-gnu target - #1095

Open
monosans wants to merge 23 commits into
mainfrom
ci/loongarch64-unknown-linux-gnu
Open

monosans wants to merge 23 commits into
mainfrom
ci/loongarch64-unknown-linux-gnu

Conversation

@monosans

Copy link
Copy Markdown
Owner

No description provided.

@monosans
monosans enabled auto-merge (squash) March 10, 2026 10:02
@coderabbitai

coderabbitai Bot commented Apr 2, 2026 •

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 7c076fc7-9ba2-4e87-9720-7663a37c74de

📥 Commits

Reviewing files that changed from the base of the PR and between acef77a and 6c99bed.

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

Summary by CodeRabbit

  • New Features
    • Added wheel builds for the LoongArch64 Linux architecture.
    • Extended code quality checks to cover LoongArch64 Linux.

Walkthrough

The CI workflow enables LoongArch64 wheel builds using ubuntu-24.04 and manylinux 2_36, and adds the target to the clippy job’s Rust target list.

Changes

CI Workflow Configuration

Layer / File(s) Summary
Enable loongarch64 in build matrix and clippy
.github/workflows/ci.yml
Activates the loongarch64-unknown-linux-gnu build matrix entry and adds the same target to the clippy job’s Rust target list.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided, so there is no meaningful description to evaluate. Add a brief description of the CI workflow change and the reason for enabling loongarch64-unknown-linux-gnu.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the CI change to add the loongarch64-unknown-linux-gnu target.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/loongarch64-unknown-linux-gnu
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch ci/loongarch64-unknown-linux-gnu

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.

This branch has not been deployed

No deployments
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