Merge #63

63: Sync CI configuration from serde_with r=jonasbb a=jonasbb bors r+ Co-authored-by: Jonas Bushart <jonas@bushart.org>
jonasbb · Mar 13, 2021 · 295a365 · 295a365
2 parents 9270afe + 3a8358b
commit 295a365
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -36,19 +36,19 @@ jobs:
         name: clippy "No Default Features" (${{ matrix.os }} / ${{ matrix.rust }})
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
-          args: --workspace --no-default-features -- -D warnings
+          args: --workspace --no-default-features --all-targets -- -D warnings
           name: clippy "No Default Features" (${{ matrix.os }} / ${{ matrix.rust }})
       - uses: actions-rs/clippy-check@v1
         name: clippy "Default" (${{ matrix.os }} / ${{ matrix.rust }})
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
-          args: --workspace -- -D warnings
+          args: --workspace --all-targets -- -D warnings
           name: clippy "Default" (${{ matrix.os }} / ${{ matrix.rust }})
       - uses: actions-rs/clippy-check@v1
         name: clippy "All Features" (${{ matrix.os }} / ${{ matrix.rust }})
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
-          args: --workspace --all-features -- -D warnings
+          args: --workspace --all-features --all-targets -- -D warnings
           name: clippy "All Features" (${{ matrix.os }} / ${{ matrix.rust }})
 
   rustfmt:
@@ -90,22 +90,25 @@ jobs:
         name: Build (${{ matrix.os }} / ${{ matrix.rust }})
         with:
           command: build
-          args: --all --all-features
+          args: --workspace --all-features --all-targets
       - uses: actions-rs/cargo@v1
         name: Test "No Default Features" (${{ matrix.os }} / ${{ matrix.rust }})
         with:
           command: test
-          args: --all --no-default-features
+          # cargo test --all-targets does NOT run doctests
+          # since doctests are important this should not be added
+          # https://github.com/rust-lang/cargo/issues/6669
+          args: --workspace --no-default-features
       - uses: actions-rs/cargo@v1
         name: Test "Default" (${{ matrix.os }} / ${{ matrix.rust }})
         with:
           command: test
-          args: --all
+          args: --workspace
       - uses: actions-rs/cargo@v1
         name: Test "All Features" (${{ matrix.os }} / ${{ matrix.rust }})
         with:
           command: test
-          args: --all --all-features
+          args: --workspace --all-features
 
   # Added to summarize the matrix (otherwise we would need to list every single
   # job in bors.toml)