Skip to content

ci: update ci build matrix#316

Merged
ViralBShah merged 14 commits intomasterfrom
ci-fix
Apr 28, 2026
Merged

ci: update ci build matrix#316
ViralBShah merged 14 commits intomasterfrom
ci-fix

Conversation

@inkydragon
Copy link
Copy Markdown
Member

@inkydragon inkydragon commented Dec 12, 2025

Test with:

  • [ubuntu-latest, windows-latest, macos-15-intel] x [x64] x ['lts', '1']
  • macos-latest x [aarch64] x ['lts', '1']
  • ubuntu-latest x [x86] x ['1']
  • (tier 2) ubuntu-24.04-arm x [aarch64] x ['1']

replace #313, fix #315

> The macOS-13 based runner images are being deprecated, consider switching to macOS-15 (macos-15-intel) or macOS 15 arm64 (macos-latest) instead
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.42%. Comparing base (cadbbc1) to head (2ad6377).
⚠️ Report is 15 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #316      +/-   ##
==========================================
+ Coverage   92.92%   96.42%   +3.50%     
==========================================
  Files           7        7              
  Lines         735      784      +49     
==========================================
+ Hits          683      756      +73     
+ Misses         52       28      -24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread .github/workflows/UnitTest.yml Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR consolidates ARM architecture testing by removing the separate UnitTestArm.yml workflow and integrating ARM testing into the main UnitTest.yml workflow. The changes modernize the CI configuration by adding concurrency controls, updating Julia version specifiers, and attempting to use native ARM runners instead of emulation.

Key Changes:

  • Removes separate ARM workflow that used Docker-based emulation via uraimo/run-on-arch-action
  • Adds concurrency configuration to cancel redundant PR builds
  • Updates Julia version testing from '1.6' to 'lts' and attempts to add native ARM runner support

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
.github/workflows/UnitTestArm.yml Deleted - separate ARM workflow using Docker emulation removed
.github/workflows/UnitTest.yml Updated to consolidate testing, add concurrency controls, attempt ARM runner integration, and modernize Julia version specs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/UnitTest.yml
Comment thread .github/workflows/UnitTest.yml
Comment thread .github/workflows/UnitTest.yml
Comment thread .github/workflows/UnitTest.yml Outdated
Comment on lines +35 to +38
- os: ubuntu-24.04-arm
julia-version: 'lts'
julia-arch: aarch64
- os: ubuntu-24.04-arm
Copy link

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The runner name 'ubuntu-24.04-arm' does not appear to be a standard GitHub-hosted runner. GitHub does not currently provide ARM-based Ubuntu runners as standard hosted runners. If ARM testing on Ubuntu is required, you would typically need to use self-hosted runners or alternative solutions. Consider using 'ubuntu-latest' with emulation (via setup-julia's arch parameter), or set up self-hosted ARM runners. If this is intended to be a self-hosted runner, ensure it's properly configured in your repository settings.

Suggested change
- os: ubuntu-24.04-arm
julia-version: 'lts'
julia-arch: aarch64
- os: ubuntu-24.04-arm
- os: ubuntu-latest
julia-version: 'lts'
julia-arch: aarch64
- os: ubuntu-latest

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/UnitTest.yml
@inkydragon inkydragon changed the title ci: Update CI config ci: update ci build matrix Dec 12, 2025
inkydragon and others added 2 commits December 12, 2025 20:07
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ViralBShah
Copy link
Copy Markdown
Member

@copilot resolve the merge conflicts in this pull request

Co-authored-by: ViralBShah <744411+ViralBShah@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 28, 2026

@copilot resolve the merge conflicts in this pull request

Merge conflicts resolved in commit 145b924. The two conflicts were:

  1. .github/workflows/UnitTest.yml — kept our branch's steps (cache@v2, julia-buildpkg@v1, julia-runtest@v1)
  2. .github/workflows/UnitTestArm.yml — kept the deletion (ARM testing is consolidated into the main workflow)

Copilot AI requested a review from ViralBShah April 28, 2026 17:18
@ViralBShah ViralBShah merged commit a0e7b81 into master Apr 28, 2026
13 checks passed
@ViralBShah ViralBShah deleted the ci-fix branch April 28, 2026 19:37
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.

broken Arm CI

4 participants