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>