matrix

microsoft · Apr 16, 2024 · f2f8442 · f2f8442
1 parent cbc470f
commit f2f8442
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 4 deletions.
diff --git a/.github/workflows/clippy.yml b/.github/workflows/clippy.yml
@@ -15,11 +15,20 @@ jobs:
   cargo_clippy:
     name: Check
     runs-on: windows-2019
+
+    strategy:
+      matrix:
+        include:
+          - version: nightly
+            target: x86_64-pc-windows-msvc
+
     steps:
       - name: Checkout
         uses: actions/checkout@v4
       - name: Update toolchain
-        run: rustup update --no-self-update nightly && rustup default nightly-x86_64-pc-windows-msvc
+        run: rustup update --no-self-update ${{ matrix.version }} && rustup default ${{ matrix.version }}-${{ matrix.target }}
+      - name: Add toolchain target
+        run: rustup target add ${{ matrix.target }}
       - name: Install clippy
         run: rustup component add clippy
       - name: Fix environment

diff --git a/crates/tools/yml/src/main.rs b/crates/tools/yml/src/main.rs
@@ -86,7 +86,7 @@ jobs:
 }
 
 fn clippy_yml() {
-    let mut yml = r#"name: clippy
+    let mut yml = r"name: clippy
 
 on:
   pull_request:
@@ -103,15 +103,24 @@ jobs:
   cargo_clippy:
     name: Check
     runs-on: windows-2019
+
+    strategy:
+      matrix:
+        include:
+          - version: nightly
+            target: x86_64-pc-windows-msvc
+
     steps:
       - name: Checkout
         uses: actions/checkout@v4
       - name: Update toolchain
-        run: rustup update --no-self-update nightly && rustup default nightly-x86_64-pc-windows-msvc
+        run: rustup update --no-self-update ${{ matrix.version }} && rustup default ${{ matrix.version }}-${{ matrix.target }}
+      - name: Add toolchain target
+        run: rustup target add ${{ matrix.target }}
       - name: Install clippy
         run: rustup component add clippy
       - name: Fix environment
-        uses: ./.github/actions/fix-environment"#
+        uses: ./.github/actions/fix-environment"
         .to_string();
 
     // This unrolling is required since "cargo clippy --all" consumes too much memory for the GitHub hosted runners.