From d8695da1d17032d785f9e43502757dc5aa9aa109 Mon Sep 17 00:00:00 2001
From: Tamo <tamo@meilisearch.com>
Date: Mon, 21 Jun 2021 19:54:43 +0200
Subject: [PATCH] improve the ci

---
 .github/workflows/rust.yml | 75 ++++++++++++++++++++++++++++++++++++++
 .github/workflows/test.yml | 63 --------------------------------
 2 files changed, 75 insertions(+), 63 deletions(-)
 create mode 100644 .github/workflows/rust.yml
 delete mode 100644 .github/workflows/test.yml

diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
new file mode 100644
index 000000000..02efb7cd5
--- /dev/null
+++ b/.github/workflows/rust.yml
@@ -0,0 +1,75 @@
+name: Rust
+
+on:
+  push:
+    branches: [ staging, trying ]
+  pull_request:
+    branches: [ main ]
+
+env:
+  CARGO_TERM_COLOR: always
+
+jobs:
+  tests:
+    name: Tests on ${{ matrix.os }} with ${{ matrix.rust }}
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-18.04, macos-latest]
+        rust:
+          - stable
+          - beta
+          - nightly
+    steps:
+    - uses: actions/checkout@v2
+    - uses: actions-rs/toolchain@v1
+      with:
+        profile: minimal
+        toolchain: ${{ matrix.rust }}
+        override: true
+    - name: Run cargo check
+      uses: actions-rs/cargo@v1
+      with:
+        command: check
+        args: --all --locked
+    - name: Run cargo test
+      uses: actions-rs/cargo@v1
+      with:
+        command: test
+        args: --locked --release
+
+  # We don't run test on Windows since we get the following error: There is not enough space on the disk.
+  check-on-windows:
+    name: Cargo check on Windows
+    runs-on: windows-latest
+    steps:
+    - uses: actions/checkout@v2
+    - name: Run cargo check without any default features
+      uses: actions-rs/cargo@v1
+      with:
+        command: check
+        args: --all --locked
+    - name: Run cargo check with all default features
+      uses: actions-rs/cargo@v1
+      with:
+        command: check
+
+  fmt:
+    name: Run Rustfmt
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions-rs/toolchain@v1
+        with:
+          profile: minimal
+          toolchain: nightly
+          override: true
+          components: rustfmt
+      - name: Run cargo fmt
+        run: |
+          # Since we never ran the `build.rs` script in the benchmark directory we are missing one auto-generated import file.
+          # Since we want to trigger (and fail) this action as fast as possible, instead of building the benchmark crate
+          # we are going to create an empty file where rustfmt expect it.
+          echo -ne "\n" > benchmarks/benches/datasets_paths.rs
+          cargo fmt --all -- --check
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
deleted file mode 100644
index 4ee17a239..000000000
--- a/.github/workflows/test.yml
+++ /dev/null
@@ -1,63 +0,0 @@
-name: Continuous integration
-
-on:
-  push:
-    branches: [ staging, trying ]
-  pull_request:
-    branches: [ main ]
-
-jobs:
-  ci:
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        rust:
-          - stable
-          - beta
-          # We temporarily stop building on nightly just to fix this issue
-          # https://github.com/bheisler/TinyTemplate/pull/17
-          # Reenable it when the fix has been merged.
-          # - nightly
-
-    steps:
-      - uses: actions/checkout@v2
-
-      - uses: actions-rs/toolchain@v1
-        with:
-          profile: minimal
-          toolchain: ${{ matrix.rust }}
-          override: true
-          components: rustfmt, clippy
-
-      - uses: actions-rs/cargo@v1
-        with:
-          command: check
-          args: --all --locked
-
-      - uses: actions-rs/cargo@v1
-        with:
-          command: build
-
-      - uses: actions-rs/cargo@v1
-        with:
-          command: test
-
-      - uses: actions-rs/cargo@v1
-        with:
-          command: bench
-          args: --no-run -p benchmarks
-
-      # - uses: actions-rs/cargo@v1
-      #   with:
-      #     command: bench
-      #     args: --no-run
-
-      # - uses: actions-rs/cargo@v1
-      #   with:
-      #     command: fmt
-      #     args: -- --check
-
-      # - uses: actions-rs/cargo@v1
-      #   with:
-      #     command: clippy
-      #     args: --all-targets -- -D warnings