Fix deploy-results being skipped due to needs chain skip propagation

Add if: always() && needs.<job>.result == 'success' to all deploy/publish
jobs to break the status propagation from skipped build-image through
the always() generate-artifacts/build jobs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: SukuWc

.github/workflows/electrical_workflow.yml

@@ -96,7 +96,7 @@ jobs:
 
   deploy-results:
     needs: generate-artifacts
-    if: github.ref == 'refs/heads/main'
+    if: always() && needs.generate-artifacts.result == 'success' && github.ref == 'refs/heads/main'
     uses: ./.github/workflows/_deploy_nightly.yml
     with:
       release_name: "Nightly Electrical"

.github/workflows/firmware_workflow.yml

@@ -35,8 +35,8 @@ jobs:
     uses: ./.github/workflows/_firmware_build.yml
 
   publish-production:
-    if: needs.build.outputs.release_version != ''
     needs: build
+    if: always() && needs.build.result == 'success' && needs.build.outputs.release_version != ''
     uses: ./.github/workflows/_firmware_release_production.yml
     with:
       product_name: "knot"
@@ -46,8 +46,8 @@ jobs:
 
   publish-nightly:
     # Run on main branch OR when a tag exists to ensure nightly is never behind production releases
-    if: github.ref == 'refs/heads/main' || needs.build.outputs.release_version != ''
     needs: build
+    if: always() && needs.build.result == 'success' && (github.ref == 'refs/heads/main' || needs.build.outputs.release_version != '')
     uses: ./.github/workflows/_firmware_release_nightly.yml
     with:
       product_name: "knot"

.github/workflows/mechanical_workflow.yml

@@ -91,7 +91,7 @@ jobs:
 
   deploy-results:
     needs: generate-artifacts
-    if: github.ref == 'refs/heads/main'
+    if: always() && needs.generate-artifacts.result == 'success' && github.ref == 'refs/heads/main'
     uses: ./.github/workflows/_deploy_nightly.yml
     with:
       release_name: "Nightly Mechanical"