Merge pull request #192 from daithihearn/sonar-auto

test: switching to automatic sonar analysis
daithihearn · Oct 3, 2023 · 18d9327 · 18d9327
2 parents 91c5de7 + 5b2e299
commit 18d9327
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/.github/workflows/unit-tests.yml b/.github/workflows/unit-tests.yml
@@ -21,8 +21,19 @@ jobs:
               run: yarn install --frozen-lockfile
             - name: Test and coverage
               run: yarn jest --coverage
+            - name: Sonar Args
+              id: sonar-args
+              run: |
+                  if [[ ${{ github.ref }} == 'refs/heads/main' ]]; then
+                    echo "SONAR_ARGS=-Dsonar.branch.name=main" >> $GITHUB_OUTPUT
+                  else
+                    echo "SONAR_ARGS=-Dsonar.branch.target=main -Dsonar.branch.name=${{ github.head_ref }}" >> $GITHUB_OUTPUT
+                  fi
+
             - name: SonarCloud Scan
-              uses: SonarSource/sonarcloud-github-action@master
+              uses: sonarsource/sonarcloud-github-action@master
               env:
-                  GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any
+                  GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
                   SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+              with:
+                  args: ${{ steps.sonar-args.outputs.SONAR_ARGS }}