Skip to content

Commit fd8b950

Browse files
cloudbridgeuyclaude
andcommitted
fix(ci): restore multi-OS validation with cargo check
Use cargo check on native platforms (Linux, macOS, Windows) instead of cross-compilation. This is necessary because headless_chrome has native dependencies that don't cross-compile. Using cargo check instead of full builds provides: - Platform validation for native dependencies - Much faster than full builds (no codegen/linking) - Excludes xtask which is development-only 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 4c0d1a7 commit fd8b950

1 file changed

Lines changed: 14 additions & 13 deletions

File tree

.github/workflows/ci.yml

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -50,25 +50,18 @@ jobs:
5050
- name: Check formatting
5151
run: cargo fmt -- --check
5252

53-
cross-platform-check:
54-
name: Cross-Platform Validation
55-
runs-on: ubuntu-latest
53+
build:
54+
name: Build Check
55+
runs-on: ${{ matrix.os }}
5656
strategy:
5757
matrix:
58-
target:
59-
- x86_64-unknown-linux-gnu
60-
- aarch64-unknown-linux-gnu
61-
- x86_64-apple-darwin
62-
- aarch64-apple-darwin
63-
- x86_64-pc-windows-msvc
58+
os: [ubuntu-latest, macos-latest, windows-latest]
6459
steps:
6560
- name: Checkout code
6661
uses: actions/checkout@v5
6762

6863
- name: Install Rust toolchain
6964
uses: dtolnay/rust-toolchain@stable
70-
with:
71-
targets: ${{ matrix.target }}
7265

7366
- name: Cache cargo registry and index
7467
uses: actions/cache@v4
@@ -81,8 +74,16 @@ jobs:
8174
restore-keys: |
8275
${{ runner.os }}-cargo-
8376
84-
- name: Check compilation for ${{ matrix.target }}
85-
run: cargo check --target ${{ matrix.target }} --workspace --exclude xtask --verbose
77+
- name: Cache target directory
78+
uses: actions/cache@v4
79+
with:
80+
path: target/
81+
key: ${{ runner.os }}-target-${{ hashFiles('**/Cargo.lock') }}
82+
restore-keys: |
83+
${{ runner.os }}-target-
84+
85+
- name: Check compilation
86+
run: cargo check --workspace --exclude xtask --verbose
8687

8788
unused-deps:
8889
name: Check Unused Dependencies

0 commit comments

Comments
 (0)