Skip to content

Commit 4d93139

Browse files
committed
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.
1 parent 3297267 commit 4d93139

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

.github/workflows/drift-detection.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,7 @@ jobs:
106106
- name: Check drift detection result
107107
if: steps.drift.outputs.exit_code != '0'
108108
run: |
109-
echo "⚠️ Drift detected between REPOSITORIES.md and GitHub organization"
110-
echo "Review the drift report in the PR comment above"
111-
exit 0
109+
echo "❌ Transfer blocked: worlddriven lacks admin permission on source repository"
110+
echo "Review the drift report in the workflow summary above"
111+
echo "Grant worlddriven admin access to the source repository to unblock"
112+
exit 1

0 commit comments

Comments
 (0)