Add unsend-features to CI tests

whisperfish · Jan 12, 2024 · 6124afe · 6124afe
1 parent 097663d
commit 6124afe
Showing 1 changed file with 18 additions and 1 deletion.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -23,12 +23,17 @@ jobs:
       fail-fast: false
       matrix:
         project: ["libsignal-service-actix", "libsignal-service-hyper", "libsignal-service"]
+        features: ["", "unsend-futures"]
+        exclude:
+          # -actix always has unsend futures, so we don't have that feature flag
+          - project: "libsignal-service-actix"
+            features: "unsend-futures"
     steps:
       - uses: actions/checkout@v3
       - uses: actions-rs/clippy-check@v1
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
-          args: --manifest-path ${{ matrix.project }}/Cargo.toml
+          args: --features "${{ features }}" --manifest-path ${{ matrix.project }}/Cargo.toml
 
   build:
     name: Build (${{ matrix.project }}, Rust ${{ matrix.toolchain }})
@@ -39,13 +44,25 @@ jobs:
         project: ["libsignal-service-actix", "libsignal-service-hyper", "libsignal-service"]
         toolchain: ["stable", "beta", "nightly"]
         coverage: [false, true]
+        features: ["", "unsend-futures"]
         exclude:
+          # Coverage related excludes
           - toolchain: stable
             coverage: true
           - toolchain: beta
             coverage: true
           - toolchain: nightly
             coverage: false
+
+          # Feature flag related excludes
+            # Actix like above
+          - project: "libsignal-service-actix"
+            features: "unsend-futures"
+            # We don't need to spawn this many jobs to see that unsend-futures works
+          - features: "unsend-futures"
+            toolchain: "beta"
+          - features: "unsend-futures"
+            toolchain: "nightly"
         include:
           - project: "libsignal-service-actix"
             toolchain: "1.70"