Merge pull request #325 from rsksmart/bugfix/skip_sonarqube_forks

Bugfix: Skipping SonarQube analysis on prs from forked repos
rsksmart · Oct 17, 2024 · 4828324 · 4828324
2 parents 62140b4 + ebfffbf
commit 4828324
Showing 1 changed file with 16 additions and 11 deletions.
diff --git a/.github/workflows/build_and_test.yml b/.github/workflows/build_and_test.yml
@@ -303,19 +303,24 @@ jobs:
           GH_PR_HEAD_REF: ${{ github.head_ref }}
           GH_REF: ${{ github.ref }}
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+          IS_FORK: ${{ github.event.pull_request.head.repo.fork }}
         run: |
           chmod +x gradlew
-          if [ "$GH_EVENT" = pull_request ] && [ "${{ github.event.pull_request.head.repo.fork }}" != "true" ]; then
-            ./gradlew sonarqube --warning-mode all --no-daemon --stacktrace --info -x build -x test \
-              -Dsonar.pullrequest.base="$GH_PR_BASE_REF" \
-              -Dsonar.pullrequest.branch="$GH_PR_HEAD_REF" \
-              -Dsonar.pullrequest.key="$GH_PR_NUMBER" \
-              -Dsonar.organization=rsksmart \
-              -Dsonar.projectKey=rsksmart_powpeg-node \
-              -Dsonar.host.url="https://sonarcloud.io" \
-              -Dsonar.junit.reportPaths=build/test-results/ \
-              -Dsonar.coverage.jacoco.xmlReportPaths=build/reports/jacoco/test/jacocoTestReport.xml \
-              -Dsonar.token="$SONAR_TOKEN"
+          if [ "$GH_EVENT" = "pull_request" ]; then
+            if [ "$IS_FORK" != "true" ]; then
+              ./gradlew sonarqube --warning-mode all --no-daemon --stacktrace --info -x build -x test \
+                -Dsonar.pullrequest.base="$GH_PR_BASE_REF" \
+                -Dsonar.pullrequest.branch="$GH_PR_HEAD_REF" \
+                -Dsonar.pullrequest.key="$GH_PR_NUMBER" \
+                -Dsonar.organization=rsksmart \
+                -Dsonar.projectKey=rsksmart_powpeg-node \
+                -Dsonar.host.url="https://sonarcloud.io" \
+                -Dsonar.junit.reportPaths=build/test-results/ \
+                -Dsonar.coverage.jacoco.xmlReportPaths=build/reports/jacoco/test/jacocoTestReport.xml \
+                -Dsonar.token="$SONAR_TOKEN"
+            else
+              echo "Skipping SonarQube analysis for pull request from a forked repo."
+            fi
           else
             ./gradlew sonarqube --warning-mode all --no-daemon --stacktrace --info -x build -x test \
               -Dsonar.branch.name="$GH_REF" \