just skip unless main

.github/workflows/tests.yml

@@ -209,9 +209,13 @@ jobs:
 
       - name: SonarCloud Scan
         uses: sonarsource/sonarcloud-github-action@master
-        # thought about just skipping dependabot, but skipping all pull requests seems better, since none of them will have access to sonarcloud
+        # thought about just skipping dependabot
         # if: ${{ github.actor != 'dependabot[bot]' }}
-        if: ${{ github.event_name != 'pull_request' }}
+        # but figured all pull requests seems better, since none of them will have access to sonarcloud.
+        # however, with just skipping pull requests, the build associated with "Triggered via push" is also associated with the pull request and also fails hitting sonarcloud
+        # if: ${{ github.event_name != 'pull_request' }}
+        # so just skip everything but main
+        if: github.ref_name == 'main'
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}