From 94661e42ae079ccd82aaec31aca271c43b9de013 Mon Sep 17 00:00:00 2001
From: Edward Amsden <edward@blackriversoft.com>
Date: Tue, 21 Nov 2023 13:27:25 -0600
Subject: [PATCH] github: update ares-shared workflow to use nix-develop

---
 .github/workflows/ares-shared.yml | 38 +++++++++++++++++++------------
 1 file changed, 24 insertions(+), 14 deletions(-)

diff --git a/.github/workflows/ares-shared.yml b/.github/workflows/ares-shared.yml
index ac35f4b..a458977 100644
--- a/.github/workflows/ares-shared.yml
+++ b/.github/workflows/ares-shared.yml
@@ -3,10 +3,11 @@ on:
 
 env:
   CARGO_TERM_COLOR: always
+  ACTIONS_ALLOW_UNSECURE_COMMANDS: "true"
 
 defaults:
   run:
-    working-directory: rust/ares
+    working-directory: rust
 
 jobs:
   build:
@@ -14,29 +15,38 @@ jobs:
     steps:
       - uses: actions/checkout@v3
 
-      # Makes it easier to determine why CI might differ from local linter/build
-      #
-      - name: Version
-        run: rustc --version
+      # Install nix, required by nix-shell-action
+      - uses: cachix/install-nix-action@v23
+        name: Install Nix
+        with:
+          extra_nix_config: "extra-experimental-features = nix-command flakes"
 
+      # Check formatting
       - name: Format
-        run: cargo fmt --check
+        run: |
+          cd ares
+          nix develop --command bash -c "cargo fmt --check"
 
       # See clippy linter docs: https://github.com/rust-lang/rust-clippy
       #
       # First linter is set to fail for all warnings, then ignored warnings are
       # explicitly listed
       #
+      # XX TODO make a script with all the flags for the linter
       - name: Lint
         run: |
-          cargo clippy \
-            --all-targets \
-            --no-deps \
-            -- -D warnings \
-            -A clippy::missing_safety_doc
+          cd ares
+          nix develop --command bash -c "cargo clippy --all-targets --no-deps -- -D warnings -A clippy::missing_safety_doc"
 
+      # Build Ares
       - name: Build
-        run: cargo build --release --verbose --features check_all
+        run: |
+          cd ares
+          nix develop --command bash -c "cargo build --release --verbose --features check_all"
+
+      # Run tests
+      - name: Test
+        run: |
+          cd ares
+          nix develop --command bash -c "cargo test --verbose -- --test-threads=1"
 
-      - name: Run tests
-        run: cargo test --verbose -- --test-threads=1