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

leerho pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/datasketches-java.git


The following commit(s) were added to refs/heads/master by this push:
     new 72f5383  reconfigure maven.yml with toolchains.
72f5383 is described below

commit 72f538358e1e1f7ceae3d26803ac1b287277ccc6
Author: Lee Rhodes <[email protected]>
AuthorDate: Sun Jan 31 12:30:09 2021 -0800

    reconfigure maven.yml with toolchains.
    
    Added Coveralls token, figure out how to remove later.
---
 .github/workflows/.toolchains.xml | 13 +++++++++++++
 .github/workflows/maven.yml       | 29 +++++++++++++++++++----------
 pom.xml                           | 16 +++++++++++-----
 3 files changed, 43 insertions(+), 15 deletions(-)

diff --git a/.github/workflows/.toolchains.xml 
b/.github/workflows/.toolchains.xml
new file mode 100644
index 0000000..a7d1161
--- /dev/null
+++ b/.github/workflows/.toolchains.xml
@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF8"?>
+<toolchains>
+    <toolchain>
+        <type>jdk</type>
+        <provides>
+            <version>8</version>
+            <vendor>openjdk</vendor>
+        </provides>
+        <configuration>
+            <jdkHome>${JAVA_HOME}</jdkHome>
+        </configuration>
+    </toolchain>
+</toolchains>
\ No newline at end of file
diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml
index 311275b..d5d6d68 100644
--- a/.github/workflows/maven.yml
+++ b/.github/workflows/maven.yml
@@ -11,23 +11,32 @@ env:
 
 jobs:
     build:
-        name: Build Job
+        name: Build, Test, Coverage
         runs-on: ubuntu-latest
         
         steps:
         - name: Checkout
           uses: actions/checkout@v2
 
-        - name: Set up JDK 1.8
-          uses: actions/setup-java@v1
+        - name: Cache local Maven repository
+          uses: actions/cache@v2
           with:
-              java-version: 1.8  #OR 8.0.212 | 16.0.0-ea.28 (.28 is build #)
+            path: ~/.m2/repository
+            key: build-${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            restore-keys: |
+              build-${{ runner.os }}-maven-
 
-        - name: Maven clean test
-          run: mvn clean test jacoco:report -q -Dgpg.skip=true 
-DisDebugEnabled=true --batch-mode --file=pom.xml
+        - name: Install JDK
+          uses: AdoptOpenJDK/install-jdk@v1
+          with:
+              version: '8'
+              architecture: x64
+              impl: hotspot
+              targets: 'JDK_8;JAVA_HOME'
+
+        - name: Install Dependencies
+          run: mvn install -DskipTests=true -Dgpg.skip=true -B -V --toolchains 
.github/workflows/.toolchains.xml
 
-        - name: Coveralls
+        - name: Test & Report
           if: ${{ success() }}
-          uses: coverallsapp/github-action@master
-          with:
-              github-token: ${{ secrets.github_token }}
\ No newline at end of file
+          run: mvn test coveralls:report -Dgpg.skip=true -B -V --toolchains 
.github/workflows/.toolchains.xml
diff --git a/pom.xml b/pom.xml
index c5a10b2..867dcd2 100644
--- a/pom.xml
+++ b/pom.xml
@@ -348,24 +348,30 @@ under the License.
           <version>${jacoco-maven-plugin.version}</version>
           <executions>
             <execution>
-              <id>prepare-agent</id>
+              <id>default-prepare-agent</id>
               <goals>
                 <goal>prepare-agent</goal>
               </goals>
             </execution>
+            <execution>
+              <id>default-report</id>
+              <goals>
+                <goal>report</goal>
+              </goals>
+            </execution>  
           </executions>
         </plugin>
-        <!--
+
         <plugin>
-          # Submit code coverage report to Coveralls.io. 
+          <!-- Submit code coverage report to Coveralls.io. -->
           <groupId>org.eluder.coveralls</groupId>
           <artifactId>coveralls-maven-plugin</artifactId>
           <version>${coveralls-maven-plugin.version}</version>
           <configuration>
-            # Since we use Travis CI we do not have to put a Coveralls token 
here. 
+            <repoToken>z2NyvKNpclvuc31m2yfZrpzrPr1cnLjSd</repoToken>
           </configuration>
         </plugin>
-      -->
+
       </plugins>
     </pluginManagement>
     <plugins>


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to