Skip to content

Commit

Permalink
Remove checkstyle.violation.ignore
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
slawekjaranowski committed Jun 12, 2024
1 parent 2901779 commit fa5cec8
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -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>

Expand Down

0 comments on commit fa5cec8

Please sign in to comment.