Skip to content

ci: build and test on linux-aarch64 - #115

Merged
olantwin merged 1 commit into
mainfrom
feat/linux-aarch64
Oct 17, 2026
Merged

olantwin merged 1 commit into
mainfrom
feat/linux-aarch64

Conversation

@olantwin

@olantwin olantwin commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

🤖 AI text below 🤖

Adds linux-aarch64 to the workspace platforms and locks it. A new build-linux-aarch64 job runs the same tasks as build-and-test on an ubuntu-24.04-arm runner, and "All checks passed" waits for both. The linux-64 job is unchanged.

The SHiP dependencies resolve from the new linux-aarch64 builds on ship-experiment/stable (ShipSoft/ship-conda-recipes#215), and the rest comes from conda-forge, which now has phlex and GeoModel for aarch64. The lock was regenerated with pixi 0.82 and only adds entries. No linux-64 package changes.

This is the first build of this repo on ARM, so the new job is the actual test.

Summary by CodeRabbit

  • Build & Validation
    • Automated checks now include Linux ARM64 builds and tests alongside the existing Linux checks. This expands platform coverage and helps identify compatibility issues on ARM-based Linux systems before changes are accepted. The overall check also reports failure if either platform’s required build or tests fail or are cancelled.

Add linux-aarch64 to the workspace platforms and lock it. A new
build-linux-aarch64 job runs the same tasks on an ubuntu-24.04-arm
runner, and "All checks passed" now waits for it too. The existing
linux-64 job is unchanged.

Assisted-by: Claude Code:claude-opus-5-5
@coderabbitai

coderabbitai Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ad4a3f75-0259-4121-9f95-cab575128cd4
📥 Commits

Reviewing files that changed from the base of the PR and between 956472e and 93e64a0.

⛔ Files ignored due to path filters: 1
  • pixi.lock is excluded by !**/*.lock
📒 Files selected for processing: 2
  • .github/workflows/build-test.yml
  • pixi.toml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

The Pixi workspace now targets Linux ARM64. The CI workflow runs the existing build workflow on an ARM64 runner and includes that job in the all-checks dependencies.

Changes: Linux ARM64 build validation

Layer / File(s) Summary
Add Linux ARM64 workspace platform
pixi.toml
The workspace platform list adds linux-aarch64 and retains linux-64.
Run ARM64 builds in CI
.github/workflows/build-test.yml
The workflow adds an ARM64 job that runs the test and geometry tasks. The all-checks job now depends on both build jobs.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to 93e64

The ARM64 build and required status check extend CI coverage. No concrete merge-blocking behavior is established, so the change is ready to merge subject to ordinary checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: adding CI build and test coverage for Linux AArch64.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@olantwin
olantwin merged commit 3294e1e into main Oct 17, 2026
8 checks passed
@olantwin
olantwin deleted the feat/linux-aarch64 branch October 17, 2026 20:42
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