IDEMPIERE-4360 turn off sonar at PR because not yet support (#439)
* IDEMPIERE-4360 add sonarcloud.io * IDEMPIERE-4360 add sonarcloud.io (update properties) * IDEMPIERE-4360 add sonarcloud.io (optimize) 1. when use matrix for codeql-action then job of javascript run same job of java (mvn, sonar) so don't use matrix do define separate job let javascript job faster a lot 2. reduce maven log and sonar log from info to warn 3. move project info from pom to yml let fork central place customize CI * IDEMPIERE-4360 turn off sonar at PR because not yet support https://stackoverflow.com/a/39720346
This commit is contained in:
parent
02ce1f3c1f
commit
fb053003a9
|
@ -33,20 +33,12 @@ jobs:
|
||||||
with:
|
with:
|
||||||
java-version: '${{ env.java }}'
|
java-version: '${{ env.java }}'
|
||||||
|
|
||||||
|
# on case PR it check out to commit is merger of PR to base (master)
|
||||||
- name: Checkout repository
|
- name: Checkout repository
|
||||||
uses: actions/checkout@v2
|
uses: actions/checkout@v2
|
||||||
with:
|
with:
|
||||||
# fetch all commit so sornar can know who change a line, it's resolved Warning: Shallow clone detected, no blame information will be provided. You can convert to non-shallow with 'git fetch --unshallow'.
|
# fetch all commit so sornar can know who change a line, it's resolved Warning: Shallow clone detected, no blame information will be provided. You can convert to non-shallow with 'git fetch --unshallow'.
|
||||||
fetch-depth: 0
|
fetch-depth: 0
|
||||||
if: github.event_name != 'pull_request'
|
|
||||||
|
|
||||||
- name: Checkout repository
|
|
||||||
uses: actions/checkout@v2
|
|
||||||
with:
|
|
||||||
ref: ${{ github.event.pull_request.head.sha }} # check out PR head
|
|
||||||
# fetch all commit so sornar can know who change a line, it's resolved Warning: Shallow clone detected, no blame information will be provided. You can convert to non-shallow with 'git fetch --unshallow'.
|
|
||||||
fetch-depth: 0
|
|
||||||
if: github.event_name == 'pull_request'
|
|
||||||
|
|
||||||
- name: Cache material
|
- name: Cache material
|
||||||
uses: actions/cache@v2
|
uses: actions/cache@v2
|
||||||
|
@ -82,16 +74,24 @@ jobs:
|
||||||
# make bootstrap
|
# make bootstrap
|
||||||
# make release
|
# make release
|
||||||
|
|
||||||
|
# run sonar on master only because sonar for PR come from other repository isn't support at moment (already on develop)
|
||||||
|
# https://stackoverflow.com/a/39720346
|
||||||
|
# https://docs.sonarqube.org/latest/analysis/pr-decoration/
|
||||||
- name: Build and sonar analyze
|
- name: Build and sonar analyze
|
||||||
env:
|
env:
|
||||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any
|
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any
|
||||||
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
|
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
|
||||||
run: |
|
run: |
|
||||||
export MAVEN_OPTS="-Xmx5G -Dorg.slf4j.simpleLogger.defaultLogLevel=warn"
|
export MAVEN_OPTS="-Xmx7G -Dorg.slf4j.simpleLogger.defaultLogLevel=warn"
|
||||||
sonarProject="-Dsonar.projectKey=idempiere_idempiere -Dsonar.organization=idempiere"
|
sonarProject="-Dsonar.projectKey=idempiere_idempiere -Dsonar.organization=idempiere"
|
||||||
sonar="-Dsonar.java.source=$java -Dsonar.java.target=$java -Dsonar.host.url=https://sonarcloud.io -Dsonar.log.level=WARN"
|
sonar="-Dsonar.java.source=$java -Dsonar.java.target=$java -Dsonar.host.url=https://sonarcloud.io -Dsonar.log.level=WARN"
|
||||||
codeql="-Dfindbugs.skip -Dcheckstyle.skip -Dpmd.skip=true -Denforcer.skip -Dmaven.javadoc.skip -DskipTests -Dmaven.test.skip.exec -Dlicense.skip=true -Drat.skip=true"
|
codeql="-Dfindbugs.skip -Dcheckstyle.skip -Dpmd.skip=true -Denforcer.skip -Dmaven.javadoc.skip -DskipTests -Dmaven.test.skip.exec -Dlicense.skip=true -Drat.skip=true"
|
||||||
mvn -B -V -e $codeql $sonar $sonarProject verify org.sonarsource.scanner.maven:sonar-maven-plugin:sonar
|
mvn -B -V -e $codeql $sonar $sonarProject verify org.sonarsource.scanner.maven:sonar-maven-plugin:sonar
|
||||||
|
if: github.event_name != 'pull_request'
|
||||||
|
|
||||||
|
- name: Autobuild
|
||||||
|
uses: github/codeql-action/autobuild@v1
|
||||||
|
if: github.event_name == 'pull_request'
|
||||||
|
|
||||||
- name: Perform CodeQL Analysis
|
- name: Perform CodeQL Analysis
|
||||||
uses: github/codeql-action/analyze@v1
|
uses: github/codeql-action/analyze@v1
|
||||||
|
@ -105,13 +105,9 @@ jobs:
|
||||||
|
|
||||||
steps:
|
steps:
|
||||||
|
|
||||||
|
# on case PR it check out to commit is merger of PR to base (master)
|
||||||
- name: Checkout repository
|
- name: Checkout repository
|
||||||
uses: actions/checkout@v2
|
uses: actions/checkout@v2
|
||||||
with:
|
|
||||||
fetch-depth: 2
|
|
||||||
|
|
||||||
- run: git checkout HEAD^2
|
|
||||||
if: ${{ github.event_name == 'pull_request' }}
|
|
||||||
|
|
||||||
- name: Initialize CodeQL
|
- name: Initialize CodeQL
|
||||||
uses: github/codeql-action/init@v1
|
uses: github/codeql-action/init@v1
|
||||||
|
|
Loading…
Reference in New Issue