feat: require the validation comment on non-fork PRs

openedx · Oct 19, 2023 · 3a0a9fd · 3a0a9fd
1 parent 366f54c
commit 3a0a9fd
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/.github/workflows/validate-translation-files.yml b/.github/workflows/validate-translation-files.yml
@@ -35,11 +35,11 @@ jobs:
 
           exit $has_validation_errors
 
-      # Due to GitHub Actions security reasons this will not work on fork pull requests.
-      # This shouldn't be an issue, because bots writes directly to this repository.
       - name: Post translation validation results as a comment
-        if: always()
-        continue-on-error: true  # Don't fail the build if posting the comment fails on fork pull requests.
+        # Due to GitHub Actions security reasons this will not work on fork pull requests.
+        # This shouldn't be an issue, because bots writes directly to this repository.
+        # Don't attempt comment fails on fork pull requests.
+        if: ${{ always() && !github.event.pull_request.head.repo.fork }}
         uses: mshick/add-pr-comment@7c0890544fb33b0bdd2e59467fbacb62e028a096
         with:
           message: |