From: Paul Spooren <mail@aparcar.org>
Date: Wed, 30 Mar 2022 00:43:17 +0000 (+0100)
Subject: CI: usability improvements for tools
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=80f79beb952dcb87d967a130d326cb1dd5a077ed;p=openwrt%2Fstaging%2Fjow.git

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>
---

diff --git a/.github/workflows/tools.yml b/.github/workflows/tools.yml
index 76cbd30db7..be967833cb 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"