Cleanliness CI Fix Error & Now Can't error outside of build (#691)

MFlowCode · Nov 5, 2024 · 411cafb · 411cafb
1 parent 8aaf0d6
commit 411cafb
Showing 1 changed file with 13 additions and 8 deletions.
diff --git a/.github/workflows/cleanliness.yml b/.github/workflows/cleanliness.yml
@@ -34,42 +34,48 @@ jobs:
                 sed -i '/\/master\//d' master.txt
             
             - name: Unused Variables Diff
+              continue-on-error: true
               run: |
                 grep -F 'Wunused-variable' master.txt > mUnused.txt
                 grep -F 'Wunused-variable' pr.txt > prUnused.txt
-                diff prUnused.txt mUnused.txt || true
+                diff prUnused.txt mUnused.txt
             
             - name: Unused Dummy Arguments Diff
+              continue-on-error: true
               run: |
-                grep -F 'Wunused-dummy-argument' pr.txt > prDummy.txt
+                grep -F 'Wunused-dummy-argument' pr.txt > prDummy.txt 
                 grep -F 'Wunused-dummy-argument' master.txt > mDummy.txt
-                diff prDummy.txt mDummy.txt || true
+                diff prDummy.txt mDummy.txt
 
             - name: Unused Value Diff
+              continue-on-error: true
               run: |
                 grep -F 'Wunused-value' pr.txt > prUnused_val.txt
                 grep -F 'Wunused-value' master.txt > mUnused_val.txt
-                diff prUnused_val.txt mUnused_val.txt || true
+                diff prUnused_val.txt mUnused_val.txt
 
             - name: Maybe Uninitialized Variables Diff
+              continue-on-error: true
               run: |
                 grep -F 'Wmaybe-uninitialized' pr.txt > prMaybe.txt
                 grep -F 'Wmaybe-uninitialized' master.txt > mMaybe.txt
-                diff prMaybe.txt mMaybe.txt || true
+                diff prMaybe.txt mMaybe.txt
 
 
             - name: Everything Diff
+              continue-on-error: true
               run: |
-                grep '\-W' pr.txt > pr_every.txt 
+                grep '\-W' pr.txt > pr_every.txt
                 grep '\-W' master.txt > m_every.txt
-                diff pr_every.txt m_every.txt || true
+                diff pr_every.txt m_every.txt
 
             - name: List of Warnings
               run: |
                 cat pr_every.txt
                 
 
             - name: Summary
+              continue-on-error: true
               run: |  
                 pr_variable=$(grep -c -F 'Wunused-variable' pr.txt)
                 pr_argument=$(grep -c -F 'Wunused-dummy-argument' pr.txt)
@@ -100,6 +106,5 @@ jobs:
               if: env.pr_everything > env.master_everything 
               run: |
                 echo "Difference between warning count in PR is greater than in master."
-                exit 1