From eb804342e2965b6ee9142ef0a28c9ae93d5a6390 Mon Sep 17 00:00:00 2001
From: innovaker <66737976+innovaker@users.noreply.github.com>
Date: Thu, 20 May 2021 16:42:17 +0100
Subject: [PATCH] fix: add guard conditions for west build tests

The RAM report, ROM report and clean steps depend on the success of the board/shield build.

PR: zmkfirmware/zmk-docker#51
---
 .github/workflows/containers.yml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/.github/workflows/containers.yml b/.github/workflows/containers.yml
index cfb6307..f913d0f 100644
--- a/.github/workflows/containers.yml
+++ b/.github/workflows/containers.yml
@@ -367,11 +367,14 @@ jobs:
       - name: Test west zephyr-export
         run: west zephyr-export
       - name: Test board/shield (west build)
+        id: board-shield
         if: ${{ matrix.board != null }}
         run: west build -s app -b ${{ matrix.board }} -- ${{ matrix.shield != null && format('-DSHIELD={0}', matrix.shield) || null }}
       - name: Test RAM report (west build)
+        if: ${{ steps.board-shield.outcome == 'success' }}
         run: west build -t ram_report
       - name: Test ROM report (west build)
+        if: ${{ steps.board-shield.outcome == 'success' }}
         run: west build -t rom_report
       - name: Test west test (single)
         run: west test tests/none/normal
@@ -379,6 +382,7 @@ jobs:
         if: ${{ env.run-unit-tests == 'true' }}
         run: west test
       - name: Test clean (west build)
+        if: ${{ steps.board-shield.outcome == 'success' }}
         run: west build -t clean
       - name: Stop container
         shell: bash