From 80f79beb952dcb87d967a130d326cb1dd5a077ed Mon Sep 17 00:00:00 2001
From: Paul Spooren <mail@aparcar.org>
Date: Wed, 30 Mar 2022 01:43:17 +0100
Subject: [PATCH] CI: usability improvements for tools

* Always store build logs
* Store .config as an artifact
* Rename job to `tools-{ os }` for log archive without spaces
* Run CI job on changes to the CI file itself

Signed-off-by: Paul Spooren <mail@aparcar.org>
---
 .github/workflows/tools.yml | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/tools.yml b/.github/workflows/tools.yml
index 76cbd30db7e..be967833cb4 100644
--- a/.github/workflows/tools.yml
+++ b/.github/workflows/tools.yml
@@ -4,10 +4,11 @@ on:
   pull_request:
     paths:
       - 'tools/**'
+      - '.github/workflows/tools.yml'
 
 jobs:
   build:
-    name: Build tools on ${{ matrix.os }}
+    name: tools-${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: False
@@ -122,13 +123,22 @@ jobs:
           make tools/install -j$(nproc) BUILD_LOG=1
 
       - name: Move logs to GITHUB_WORKSPACE
-        if: failure()
+        if: always()
         run: |
           cp -r "$WORKPATH/logs" "$GITHUB_WORKSPACE"
+          cp -r "$WORKPATH/.config" "$GITHUB_WORKSPACE/config"
+
 
       - name: Upload logs
-        if: failure()
+        if: always()
         uses: actions/upload-artifact@v2
         with:
           name: ${{ matrix.os }}-logs
           path: "logs"
+
+      - name: Upload config
+        if: always()
+        uses: actions/upload-artifact@v2
+        with:
+          name: ${{ matrix.os }}-config
+          path: "config"