From 28c8e6f9ae51a6378124bebdd803a1e336059587 Mon Sep 17 00:00:00 2001
From: Graham McIntire <graham@mcintire.me>
Date: Sun, 8 Mar 2026 14:14:16 -0500
Subject: [PATCH] fix: remove redundant if: success() condition from deploy job

The if: success() might be causing issues with Forgejo's secret handling.
Jobs with 'needs' automatically wait for dependencies to succeed, making
the condition redundant.
---
 .forgejo/workflows/production.yaml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/.forgejo/workflows/production.yaml b/.forgejo/workflows/production.yaml
index 0401588a..3bf48fc0 100644
--- a/.forgejo/workflows/production.yaml
+++ b/.forgejo/workflows/production.yaml
@@ -215,7 +215,6 @@ jobs:
     runs-on: ubuntu-22.04
     # Only require ExUnit tests to pass, e2e is optional
     needs: [test-exunit]
-    if: success()
 
     steps:
       - name: Checkout code