Merge branch 'main' of github.com:sartography/spiff-workflow-webapp

This commit is contained in:
jasquat 2022-05-26 14:17:09 -04:00
commit 797116a727
1 changed files with 6 additions and 2 deletions

View File

@ -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 }}