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 <[email protected]>
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>