You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ah...@apache.org on 2019/07/22 17:04:01 UTC

[commons-rng] branch master updated (d85eb21 -> 802b4fa)

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

aherbert pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/commons-rng.git.


    from d85eb21  Remove JVM unsupported on Travis.
     new dfae53e  Run the default Maven goal.
     new fa7e448  Ensure rat plugin config matches between build and reporting sections.
     new 802b4fa  Use commons parent defaults for coverage.

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .travis.yml |  9 ++++++-
 pom.xml     | 82 ++++++++++++++++++-------------------------------------------
 2 files changed, 32 insertions(+), 59 deletions(-)


[commons-rng] 01/03: Run the default Maven goal.

Posted by ah...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-rng.git

commit dfae53e0e13f5e01640be51a9d71ac50ece01afa
Author: aherbert <ah...@apache.org>
AuthorDate: Mon Jul 22 13:28:12 2019 +0100

    Run the default Maven goal.
    
    This adds the following goals to the previous config:
    
    - apache-rat:check
    - javadoc:javadoc
    
    The build does not require a prior install step so this has been skipped.
    
    After success submit the coverage report.
---
 .travis.yml | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/.travis.yml b/.travis.yml
index 828b712..476d8e3 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -21,5 +21,12 @@ jdk:
   - openjdk8
   - oraclejdk9
 
+# Skip the installation step:
+#   https://docs.travis-ci.com/user/job-lifecycle/#skipping-the-installation-phase
+install: true
+
+script:
+  - mvn
+
 after_success:
-  - mvn clean test pmd:check spotbugs:check checkstyle:check jacoco:report coveralls:report
+  - mvn jacoco:report coveralls:report


[commons-rng] 03/03: Use commons parent defaults for coverage.

Posted by ah...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-rng.git

commit 802b4fa4f921ef10387a7ed5b519b20bdbd1db95
Author: aherbert <ah...@apache.org>
AuthorDate: Mon Jul 22 14:56:22 2019 +0100

    Use commons parent defaults for coverage.
    
    These are stricter than the overrides that have been removed:
    
    classRatio 0.96 -> 1.0
    instructionRatio 0.8 -> 0.9
    methodRatio 0.8 -> 0.95
    branchRatio 0.8 -> 0.85
    complexityRatio 0.8 -> 0.85
    lineRatio 0.85 -> 0.9
    
    Removed jacoco travis profile and use commons-parent profile.
    
    Added a pre jdk-8 profile to change the default goal to ignore JDK 8 plugins.
    These cause Travis to fail on the JDK 7 build.
---
 pom.xml | 70 ++++++++++++++---------------------------------------------------
 1 file changed, 15 insertions(+), 55 deletions(-)

diff --git a/pom.xml b/pom.xml
index 6a09468..f72d0b3 100644
--- a/pom.xml
+++ b/pom.xml
@@ -126,12 +126,6 @@
     <!-- Fix to build on JDK 7: version 4.0.0 requires Java 8. -->
     <commons.felix.version>3.5.1</commons.felix.version>
 
-    <commons.jacoco.classRatio>0.96</commons.jacoco.classRatio>
-    <commons.jacoco.instructionRatio>0.8</commons.jacoco.instructionRatio>
-    <commons.jacoco.methodRatio>0.8</commons.jacoco.methodRatio>
-    <commons.jacoco.branchRatio>0.8</commons.jacoco.branchRatio>
-    <commons.jacoco.complexityRatio>0.8</commons.jacoco.complexityRatio>
-    <commons.jacoco.lineRatio>0.85</commons.jacoco.lineRatio>
     <commons.jacoco.haltOnFailure>false</commons.jacoco.haltOnFailure>
 
     <commons.site.path>rng</commons.site.path>
@@ -510,64 +504,30 @@
             <groupId>org.jacoco</groupId>
             <artifactId>jacoco-maven-plugin</artifactId>
             <version>${commons.jacoco.version}</version>
-            <executions>
-              <execution>
-                <id>default-prepare-agent</id>
-                <goals>
-                  <goal>prepare-agent</goal>
-                </goals>
-              </execution>
-              <execution>
-                <id>default-prepare-agent-integration</id>
-                <goals>
-                  <goal>prepare-agent-integration</goal>
-                </goals>
-              </execution>
-              <execution>
-                <id>default-report</id>
-                <goals>
-                  <goal>report</goal>
-                </goals>
-              </execution>
-              <execution>
-                <id>default-report-integration</id>
-                <goals>
-                  <goal>report-integration</goal>
-                </goals>
-              </execution>
-              <execution>
-                <id>default-check</id>
-                <goals>
-                  <goal>check</goal>
-                </goals>
-                <configuration>
-                  <rules>
-                    <!--  implementation is needed only for Maven 2  -->
-                    <rule implementation="org.jacoco.maven.RuleConfiguration">
-                      <element>BUNDLE</element>
-                      <limits>
-                        <!--  implementation is needed only for Maven 2  -->
-                        <limit implementation="org.jacoco.report.check.Limit">
-                          <counter>COMPLEXITY</counter>
-                          <value>COVEREDRATIO</value>
-                          <minimum>0.60</minimum>
-                        </limit>
-                      </limits>
-                    </rule>
-                  </rules>
-                </configuration>
-              </execution>
-            </executions>
           </plugin>
           <plugin>
             <groupId>org.eluder.coveralls</groupId>
             <artifactId>coveralls-maven-plugin</artifactId>
+            <!-- Version for JDK 6 target. -->
             <version>3.1.0</version>
+            <configuration>
+              <timestampFormat>${commons.coveralls.timestampFormat}</timestampFormat>
+            </configuration>
           </plugin>
         </plugins>
       </build>
     </profile>
-    <!-- profile to allow the use of plugin versions that require Java 8+ -->
+    <!-- Profile to alter default Maven goal as it uses checks from Java 8+ plugins. -->
+    <profile>
+      <id>pre-jdk8</id>
+      <activation>
+        <jdk>[1.6,1.8)</jdk>
+      </activation>
+      <build>
+        <defaultGoal>clean verify</defaultGoal>
+      </build>
+    </profile>
+    <!-- Profile to allow the use of plugin versions that require Java 8+ -->
     <profile>
       <id>jdk8-plugins</id>
       <activation>


[commons-rng] 02/03: Ensure rat plugin config matches between build and reporting sections.

Posted by ah...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-rng.git

commit fa7e448189fa465c05a646f3efe1bb1347d21e63
Author: aherbert <ah...@apache.org>
AuthorDate: Mon Jul 22 13:33:55 2019 +0100

    Ensure rat plugin config matches between build and reporting sections.
---
 pom.xml | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/pom.xml b/pom.xml
index e3471eb..6a09468 100644
--- a/pom.xml
+++ b/pom.xml
@@ -296,13 +296,19 @@
         <groupId>org.apache.rat</groupId>
         <artifactId>apache-rat-plugin</artifactId>
         <configuration>
-         <!--  Should agree with apache-rat-plugin config under <build> -->
+          <!--  Should agree with apache-rat-plugin config under <build> -->
           <excludes combine.children="append">
             <!-- version 0.8 of apache-rat-plugin does not exclude properly
                  some default development tools files (see RAT-126) -->
             <exclude>.ekstazi/**</exclude>
-            <exclude>src/site/resources/txt/userguide/stress/dh/**</exclude>
-            <exclude>src/site/resources/txt/userguide/stress/tu/**</exclude>
+            <exclude>**/site-content/**</exclude>
+            <exclude>**/.classpath</exclude>
+            <exclude>**/.project</exclude>
+            <exclude>**/.settings/**</exclude>
+            <exclude>**/.checkstyle</exclude>
+            <exclude>**/target/**</exclude>
+            <exclude>src/site/resources/release-notes/RELEASE-NOTES-*.txt</exclude>
+            <exclude>src/site/resources/txt/userguide/stress/**</exclude>
             <exclude>dist-archive/**</exclude>
           </excludes>
         </configuration>