From 320b691086745b44f3127f4e73b95c546acafc03 Mon Sep 17 00:00:00 2001
From: ReenigneArcher <42013603+ReenigneArcher@users.noreply.github.com>
Date: Wed, 16 Feb 2022 18:09:23 -0500
Subject: [PATCH] Update clang.yml

- Use job strategy matrix
  - inplace True allows artifacts to be uploaded; however workflow succeeds even if there are errors
  - inplace False fails workflow if there are errors
---
 .github/workflows/clang.yml | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/clang.yml b/.github/workflows/clang.yml
index 6ade2d0b..1031378e 100644
--- a/.github/workflows/clang.yml
+++ b/.github/workflows/clang.yml
@@ -7,7 +7,12 @@ on:
 
 jobs:
   lint:
+    name: Clang Format Lint
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false  # false to test all, true to fail entire job if any fail
+      matrix:
+        inplace: [ True, False ]  # removed ubuntu_18_04 for now
 
     steps:
     - name: Checkout
@@ -20,10 +25,11 @@ jobs:
         extensions: 'cpp,h'
         clangFormatVersion: 13
         style: file
-        inplace: True
+        inplace: ${{ matrix.inplace }}
 
     - name: Upload Artifacts
+      if: ${{ matrix.inplace == True }}
       uses: actions/upload-artifact@v2
       with:
-        name: clang-formatted-files
+        name: sunshine
         path: sunshine/