Skip to content

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

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

  • Platform Support
    • Linux ARM64 is now included in build and test coverage alongside Linux x86-64.
    • The development workspace now targets both Linux ARM64 and Linux x86-64. This extends platform coverage for builds and automated checks.

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: 1f0dd8de-71bf-41bf-9019-6e61f7d3434b
📥 Commits

Reviewing files that changed from the base of the PR and between 1d44696 and ec924f1.

⛔ 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-and-test task on an ARM64 runner, and all-checks waits for that job.

Changes: Linux ARM64 support

Layer / File(s) Summary
Add ARM64 platform and CI validation
.github/workflows/build-test.yml, pixi.toml
The workspace platform list adds linux-aarch64. The workflow adds an ARM64 build-and-test job, and all-checks depends on both build-and-test jobs.

Priority: ⬇️ Low

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

Change: Feature

Merge Risk: ⚪ Minimal · up to ec924

The new ARM64 check uses the supported runner input and runs the configure, build, and test tasks. No concrete merge-blocking risk was established.

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 Linux ARM64 build and test coverage in CI.
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 ba0dda8 into main Oct 17, 2026
8 checks passed
@olantwin
olantwin deleted the feat/linux-aarch64 branch October 17, 2026 20:36
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