You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sj...@apache.org on 2024/06/02 09:15:00 UTC

(maven-shared-resources) branch MSHARED-1401-magic-numner created (now 68666b6)

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

sjaranowski pushed a change to branch MSHARED-1401-magic-numner
in repository https://gitbox.apache.org/repos/asf/maven-shared-resources.git


      at 68666b6  [MSHARED-1401] Checkstyle drop MagicNumber

This branch includes the following new commits:

     new 68666b6  [MSHARED-1401] Checkstyle drop MagicNumber

The 1 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.



(maven-shared-resources) 01/01: [MSHARED-1401] Checkstyle drop MagicNumber

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

sjaranowski pushed a commit to branch MSHARED-1401-magic-numner
in repository https://gitbox.apache.org/repos/asf/maven-shared-resources.git

commit 68666b636ff9193a3c6d7cab4cfdfd8422933b4a
Author: Slawomir Jaranowski <s....@gmail.com>
AuthorDate: Sun Jun 2 11:14:47 2024 +0200

    [MSHARED-1401] Checkstyle drop MagicNumber
    
    more of items are excluded now
---
 src/main/resources/config/maven_checks.xml             | 4 ----
 src/main/resources/config/maven_checks_nocodestyle.xml | 4 ----
 2 files changed, 8 deletions(-)

diff --git a/src/main/resources/config/maven_checks.xml b/src/main/resources/config/maven_checks.xml
index 240f595..04a2810 100644
--- a/src/main/resources/config/maven_checks.xml
+++ b/src/main/resources/config/maven_checks.xml
@@ -173,10 +173,6 @@ under the License.
         </module>
         <module name="IllegalInstantiation"/>
         <module name="InnerAssignment"/>
-        <module name="MagicNumber">
-          <!-- some numbers are really not that magic -->
-          <property name="ignoreNumbers" value="-4, -3, -2, -1, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 31, 32, 37, 64, 100, 128, 256, 512, 1000, 1024"/>
-        </module>
         <module name="MissingSwitchDefault"/>
         <module name="SimplifyBooleanExpression"/>
         <module name="SimplifyBooleanReturn"/>
diff --git a/src/main/resources/config/maven_checks_nocodestyle.xml b/src/main/resources/config/maven_checks_nocodestyle.xml
index 70e68e8..39e70ad 100644
--- a/src/main/resources/config/maven_checks_nocodestyle.xml
+++ b/src/main/resources/config/maven_checks_nocodestyle.xml
@@ -116,10 +116,6 @@ under the License.
         </module>
         <module name="IllegalInstantiation"/>
         <module name="InnerAssignment"/>
-        <module name="MagicNumber">
-            <!-- some numbers are really not that magic -->
-            <property name="ignoreNumbers" value="-4, -3, -2, -1, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 31, 32, 37, 64, 100, 128, 256, 512, 1000, 1024"/>
-        </module>
         <module name="MissingSwitchDefault"/>
         <module name="SimplifyBooleanExpression"/>
         <module name="SimplifyBooleanReturn"/>