Skip to content

Enhance CI workflow for multi-OS and architecture builds#32

Merged
SF-Zhou merged 4 commits intomainfrom
SF-Zhou-patch-1
Mar 9, 2026
Merged

Enhance CI workflow for multi-OS and architecture builds#32
SF-Zhou merged 4 commits intomainfrom
SF-Zhou-patch-1

Conversation

@SF-Zhou
Copy link
Copy Markdown
Owner

@SF-Zhou SF-Zhou commented Mar 9, 2026

Updated the CI workflow to support multiple OS and architectures using a matrix strategy.

Updated the CI workflow to support multiple OS and architectures using a matrix strategy.
Copilot AI review requested due to automatic review settings March 9, 2026 01:13
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (415adb1) to head (0896af5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #32   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines         1533      1533           
=========================================
  Hits          1533      1533           

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

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

Updates the Rust CI workflow to run the build job on multiple operating systems (and nominally multiple architectures) via a GitHub Actions matrix.

Changes:

  • Replaced fixed runs-on: ubuntu-latest with runs-on: ${{ matrix.os }}.
  • Added a matrix with OS/target combinations for Linux, macOS, and Windows.

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

Comment thread .github/workflows/rust.yml
Comment thread .github/workflows/rust.yml Outdated
Comment on lines +36 to +38
- build: windows-arm64
os: windows-latest
target: aarch64-pc-windows-msvc
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

windows-arm64 is configured to run on windows-latest, which is an x86_64 runner. If the intent is to validate ARM64 Windows, this will not do so unless you add an explicit cross-compilation setup (and potentially skip running tests/coverage for that target). Consider removing this entry or switching to a supported approach for ARM64 validation (cross build only, or a self-hosted ARM64 runner).

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/rust.yml Outdated
Comment on lines +27 to +29
- build: linux-arm64
os: ubuntu-latest
target: aarch64-unknown-linux-gnu
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

linux-arm64 is set to run on ubuntu-latest (x86_64). Without additional cross-compilation/QEMU setup (and without using ${{ matrix.target }} in the steps), this does not actually validate an ARM64 Linux build. Either add the required cross toolchain/configuration and a cargo build --target step, or drop this matrix entry to avoid misleading CI coverage.

Copilot uses AI. Check for mistakes.
SF-Zhou added 3 commits March 9, 2026 09:29
Updated GitHub Actions workflow to include testing jobs for multiple OS and improved coverage reporting.
@SF-Zhou SF-Zhou merged commit 2e010eb into main Mar 9, 2026
12 checks passed
@SF-Zhou SF-Zhou deleted the SF-Zhou-patch-1 branch March 9, 2026 01:45
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.

2 participants