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

sjaranowski pushed a commit to branch rm-checkstyle-ignore
in repository https://gitbox.apache.org/repos/asf/maven-parent.git

commit 70aad53629ba2ec2f984374b0ab18fdfa84c7b62
Author: Slawomir Jaranowski <s.jaranow...@gmail.com>
AuthorDate: Sun Jun 2 16:26:33 2024 +0200

    Remove checkstyle.violation.ignore
    
    Ignore rules should be defined by project, class, method ...
    or we should drop a rule from configuration at all.
    
    Ignored rules are still present in checkstyle report.
---
 pom.xml | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/pom.xml b/pom.xml
index 1bec6f7..977f753 100644
--- a/pom.xml
+++ b/pom.xml
@@ -959,9 +959,6 @@ under the License.
     <version.plexus-xml>3.0.0</version.plexus-xml>
     <versions.junit5>5.10.2</versions.junit5>
     <version.maven-fluido-skin>1.11.2</version.maven-fluido-skin>
-    <!-- don't fail check for some rules that are too hard to enforce (could 
even be told broken for some)
-         and those that are enforced by the formatting checks from spotless -->
-    
<checkstyle.violation.ignore>ParameterNumber,MethodLength,FileLength</checkstyle.violation.ignore>
     
<project.build.outputTimestamp>2024-04-13T21:16:21Z</project.build.outputTimestamp>
   </properties>
 

Reply via email to