This is an automated email from the ASF dual-hosted git repository.

jinterrante pushed a commit to branch runtime2-2202
in repository https://gitbox.apache.org/repos/asf/daffodil.git


The following commit(s) were added to refs/heads/runtime2-2202 by this push:
     new 9b35efa  Don't setup Java in SonarCloud workflow
9b35efa is described below

commit 9b35efa50e02830d356fa5c1c8e72e76f4abd307
Author: John Interrante <[email protected]>
AuthorDate: Tue Mar 30 13:17:54 2021 -0400

    Don't setup Java in SonarCloud workflow
    
    Setting up Java (which sets JAVA_HOME, etc.) will screw up the
    sonarcloud action and make it unable to find java inside its Docker
    container.
---
 .github/workflows/sonarcloud.yml | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/.github/workflows/sonarcloud.yml b/.github/workflows/sonarcloud.yml
index b20fe7d..fd5c81a 100644
--- a/.github/workflows/sonarcloud.yml
+++ b/.github/workflows/sonarcloud.yml
@@ -20,16 +20,13 @@ on: [push]
 jobs:
   sonarCloudTrigger:
     if: github.repository == 'apache/daffodil'
-    name: SonarCloud ${{ matrix.java_version }}, Scala ${{ 
matrix.scala_version }}, ${{ matrix.os }}
+    name: SonarCloud Scan, Scala ${{ matrix.scala_version }}, ${{ matrix.os }}
     strategy:
       matrix:
-        java_version: [ 11 ]
         scala_version: [ 2.12.11 ]
         os: [ ubuntu-20.04 ]
 
     runs-on: ${{ matrix.os }}
-    env:
-      SBT: sbt -J-Xms1024m -J-Xmx5120m -J-XX:ReservedCodeCacheSize=512m 
-J-XX:MaxMetaspaceSize=1024m ++${{ matrix.scala_version }}
     steps:
 
       ############################################################
@@ -40,11 +37,6 @@ jobs:
         if: runner.os == 'Linux'
         run: sudo apt-get install -y libmxml-dev
 
-      - name: Setup Java
-        uses: actions/[email protected]
-        with:
-          java-version: ${{ matrix.java_version }}
-
       - name: Check out Repository
         uses: actions/[email protected]
         # Disabling shallow clone is recommended for improving reporting
@@ -57,9 +49,12 @@ jobs:
 
       - name: Compile
         run: $SBT compile test:compile it:compile
+        env:
+          SBT: sbt -J-Xms1024m -J-Xmx5120m -J-XX:ReservedCodeCacheSize=512m 
-J-XX:MaxMetaspaceSize=1024m ++${{ matrix.scala_version }}
 
       - name: SonarCloud Scan
         uses: sonarsource/[email protected]
+        # Don't setup Java since its env vars will screw up sonarcloud action
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}

Reply via email to