You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@creadur.apache.org by po...@apache.org on 2017/10/21 18:54:12 UTC

svn commit: r1812848 - in /creadur/rat/trunk: apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java src/changes/changes.xml

Author: pottlinger
Date: Sat Oct 21 18:54:11 2017
New Revision: 1812848

URL: http://svn.apache.org/viewvc?rev=1812848&view=rev
Log:
RAT-243: Add .checkstyle to default exclusions

Modified:
    creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java
    creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java
    creadur/rat/trunk/src/changes/changes.xml

Modified: creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java
URL: http://svn.apache.org/viewvc/creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java?rev=1812848&r1=1812847&r2=1812848&view=diff
==============================================================================
--- creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java (original)
+++ creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java Sat Oct 21 18:54:11 2017
@@ -50,7 +50,8 @@ public final class ExclusionHelper {
      * The Eclipse specific default excludes.
      */
     static final List<String> ECLIPSE_DEFAULT_EXCLUDES = Collections
-            .unmodifiableList(Arrays.asList(".classpath",//
+            .unmodifiableList(Arrays.asList(".checkstyle",
+                    ".classpath",//
                     ".project", //
                     ".settings/**/*"));
 

Modified: creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java
URL: http://svn.apache.org/viewvc/creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java?rev=1812848&r1=1812847&r2=1812848&view=diff
==============================================================================
--- creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java (original)
+++ creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java Sat Oct 21 18:54:11 2017
@@ -45,7 +45,7 @@ public class ExclusionHelperTest {
 
     @Test
     public void testNumberOfExclusions() {
-        assertEquals("Did you change the number of eclipse excludes?", 3,
+        assertEquals("Did you change the number of eclipse excludes?", 4,
                 ECLIPSE_DEFAULT_EXCLUDES.size());
         assertEquals("Did you change the number of idea excludes?", 4,
                 IDEA_DEFAULT_EXCLUDES.size());
@@ -59,9 +59,9 @@ public class ExclusionHelperTest {
         addEclipseDefaults(log, false, exclusion);
         assertTrue(exclusion.isEmpty());
         addEclipseDefaults(log, true, exclusion);
-        assertEquals(3, exclusion.size());
+        assertEquals(4, exclusion.size());
         addEclipseDefaults(log, true, exclusion);
-        assertEquals(3, exclusion.size());
+        assertEquals(4, exclusion.size());
     }
 
     @Test

Modified: creadur/rat/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/creadur/rat/trunk/src/changes/changes.xml?rev=1812848&r1=1812847&r2=1812848&view=diff
==============================================================================
--- creadur/rat/trunk/src/changes/changes.xml (original)
+++ creadur/rat/trunk/src/changes/changes.xml Sat Oct 21 18:54:11 2017
@@ -55,6 +55,9 @@ The <action> type attribute can be add,u
 
   <body>
     <release version="0.13" date="2017-xx-xx" description="Current SNAPSHOT - to be done">
+      <action issue="RAT-243" type="add" due-to="Matthew Ouyang" dev="pottlinger">
+	Add .checkstyle to Eclipse default exclusions.
+      </action>
       <action issue="RAT-241" type="add" due-to="Andrew Gaul" dev="pottlinger">
 	Reduce default log level of used exclusions to debug, only print totals into the maven log like includes.
       </action>