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


The following commit(s) were added to refs/heads/rm-checkstyle-ignore by this 
push:
     new fa5cec8  Remove checkstyle.violation.ignore
fa5cec8 is described below

commit fa5cec8590fc5cea7a7870226820d37a2e0c92b8
Author: Slawomir Jaranowski <s.jaranow...@gmail.com>
AuthorDate: Wed Jun 12 23:52:22 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 852128c..7a59596 100644
--- a/pom.xml
+++ b/pom.xml
@@ -960,9 +960,6 @@ under the License.
     <versions.junit5>5.10.2</versions.junit5>
     <version.maven-fluido-skin>1.11.2</version.maven-fluido-skin>
     <version.maven-shared-resources>5</version.maven-shared-resources>
-    <!-- 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