SCSCANGHA-13 Warning instead of failure for Maven/Gradle projects

SonarSource · May 20, 2024 · 1fae13a · 1fae13a
1 parent 3da91d8
commit 1fae13a
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 19 deletions.
diff --git a/.github/workflows/qa.yml b/.github/workflows/qa.yml
@@ -63,7 +63,7 @@ jobs:
         run: |
           echo "Expected previous step to fail"
           exit 1
-  failFastGradleTest:
+  dontFailGradleTest:
     name: >
       Fail fast on Gradle project
     runs-on: ubuntu-latest
@@ -79,12 +79,11 @@ jobs:
           SONAR_TOKEN: FAKE_TOKEN
         with:
           projectBaseDir: ./test/gradle-project
-      - name: Previous should have failed
-        if: ${{ steps.runTest.outcome == 'success'}}
+          args: -Dsonar.scanner.dumpToFile=./output.properties
+      - name: Assert
         run: |
-          echo "Expected previous step to fail"
-          exit 1
-  failFastGradleKotlinTest:
+          ./test/assertFileExists ./output.properties
+  dontFailGradleKotlinTest:
     name: >
       Fail fast on Kotlin Gradle project
     runs-on: ubuntu-latest
@@ -100,12 +99,11 @@ jobs:
           SONAR_TOKEN: FAKE_TOKEN
         with:
           projectBaseDir: ./test/gradle-project-kotlin
-      - name: Previous should have failed
-        if: ${{ steps.runTest.outcome == 'success'}}
+          args: -Dsonar.scanner.dumpToFile=./output.properties
+      - name: Assert
         run: |
-          echo "Expected previous step to fail"
-          exit 1
-  failFastMavenTest:
+          ./test/assertFileExists ./output.properties
+  dontFailMavenTest:
     name: >
       Fail fast on Maven project
     runs-on: ubuntu-latest
@@ -121,8 +119,7 @@ jobs:
           SONAR_TOKEN: FAKE_TOKEN
         with:
           projectBaseDir: ./test/maven-project
-      - name: Previous should have failed
-        if: ${{ steps.runTest.outcome == 'success'}}
+          args: -Dsonar.scanner.dumpToFile=./output.properties
+      - name: Assert
         run: |
-          echo "Expected previous step to fail"
-          exit 1
+          ./test/assertFileExists ./output.properties
diff --git a/entrypoint.sh b/entrypoint.sh
@@ -8,13 +8,11 @@ if [[ -z "${SONAR_TOKEN}" ]]; then
 fi
 
 if [[ -f "${INPUT_PROJECTBASEDIR%/}/pom.xml" ]]; then
-  echo "Maven project detected. You should run the goal 'org.sonarsource.scanner.maven:sonar' during build rather than using this GitHub Action."
-  exit 1
+  echo "WARNING! Maven project detected, you might have only partial analysis. You can run the goal 'org.sonarsource.scanner.maven:sonar' during build instead of this GitHub Action."
 fi
 
 if [[ -f "${INPUT_PROJECTBASEDIR%/}/build.gradle" || -f "${INPUT_PROJECTBASEDIR%/}/build.gradle.kts" ]]; then
-  echo "Gradle project detected. You should use the SonarQube plugin for Gradle during build rather than using this GitHub Action."
-  exit 1
+  echo "WARNING! Gradle project detected, you might have only partial analysis. You can use the SonarQube plugin for Gradle during build instead of this GitHub Action."
 fi
 
 if [[ -z "${SONARCLOUD_URL}" ]]; then