From 3e8ef5c3ed3354ba094e2501865dfbce9785c2a8 Mon Sep 17 00:00:00 2001 From: Tobias Wilken Date: Thu, 4 Dec 2025 20:14:48 +0100 Subject: [PATCH] fix: actually fail CI when transfer permissions are missing The previous fix (PR #14) updated detect-drift.js to exit with code 1 when transfers are blocked due to missing permissions, but the workflow file was never updated to propagate that failure. The workflow had `exit 0` which always passed the job regardless of the script's exit code. Changed to `exit 1` so the CI properly fails when worlddriven lacks admin permission on source repositories. --- .github/workflows/drift-detection.yml | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/workflows/drift-detection.yml b/.github/workflows/drift-detection.yml index 62ab60a..06f3a0b 100644 --- a/.github/workflows/drift-detection.yml +++ b/.github/workflows/drift-detection.yml @@ -106,6 +106,7 @@ jobs: - name: Check drift detection result if: steps.drift.outputs.exit_code != '0' run: | - echo "⚠️ Drift detected between REPOSITORIES.md and GitHub organization" - echo "Review the drift report in the PR comment above" - exit 0 + echo "❌ Transfer blocked: worlddriven lacks admin permission on source repository" + echo "Review the drift report in the workflow summary above" + echo "Grant worlddriven admin access to the source repository to unblock" + exit 1