Skip to content
Snippets Groups Projects

Add Sonarcloud bindings to CI build and pom

Merged Martin Lowe requested to merge malowe/geoip-rest-api:malowe/master/sonar-ci into master
2 unresolved threads

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Frederic Gurr
  • 122 127 }
    123 128
    124 129 sh 'make compile-java'
    130
    131 withCredentials([string(credentialsId: 'sonarcloud-token-git-eca-rest-api', variable: 'SONAR_TOKEN')]) {
    132 sh 'mvn org.sonarsource.scanner.maven:sonar-maven-plugin:sonar -Dsonar.host.url=https://sonarcloud.io -Dsonar.login=${SONAR_TOKEN} -Dmaven.test.skip=true'
    133 }
    • Comment on lines 129 to +133

      I would recommend doing it like this:

      sh 'make generate-spec'
      withCredentials([string(credentialsId: 'sonarcloud-token-geoip-rest-api', variable: 'SONAR_TOKEN')]) {
        sh 'mvn clean verify org.sonarsource.scanner.maven:sonar-maven-plugin:sonar -B -Dsonar.login=${SONAR_TOKEN}'
      }
      
    • Please register or sign in to reply
  • Martin Lowe added 1 commit

    added 1 commit

    • a8922112 - Update Jenkinsfile to split build into sonar and non-sonar builds

    Compare with previous version

  • Martin Lowe added 1 commit

    added 1 commit

    • 9b878974 - Update Jenkinsfile to split build into sonar and non-sonar builds

    Compare with previous version

  • Frederic Gurr
  • Frederic Gurr
  • Frederic Gurr
  • Martin Lowe added 10 commits

    added 10 commits

    • 9b878974...6ac2acbe - 6 commits from branch eclipsefdn/it/api:master
    • cf2d8a28 - Add Sonarcloud bindings to CI build and pom
    • 02fdf031 - Add missing jacoco properties and fix project key
    • e93549c2 - Set stack build agent version to the new version with new nodejs release
    • 1cb04a30 - Update Jenkinsfile to split build into sonar and non-sonar builds

    Compare with previous version

  • Martin Lowe added 6 commits

    added 6 commits

    • 1cb04a30...4697260e - 2 commits from branch eclipsefdn/it/api:master
    • 2f0660c2 - Add Sonarcloud bindings to CI build and pom
    • d897caff - Add missing jacoco properties and fix project key
    • a453be6d - Set stack build agent version to the new version with new nodejs release
    • b8b738e5 - Update Jenkinsfile to split build into sonar and non-sonar builds

    Compare with previous version

  • Martin Lowe added 1 commit

    added 1 commit

    • 426fef35 - Fix Jenkins not pointing to proper default branch for sonar scan check

    Compare with previous version

  • Martin Lowe requested review from @fgurr

    requested review from @fgurr

  • Martin Lowe mentioned in commit 6da8e71b

    mentioned in commit 6da8e71b

  • merged

  • Please register or sign in to reply
    Loading