Skip to content

Commit

Permalink
ci: Fix Checkstyle configuration
Browse files Browse the repository at this point in the history
Update configuration to address the following warning:

```
The Checkstyle rules file could not be parsed. cannot initialize module TreeWalker - TreeWalker is not allowed as a parent of LineLength Please review 'Parent Module' section for this Check in web documentation if Check is standard. The file has been blocked for 60s.
```
  • Loading branch information
dcalhoun committed Jul 8, 2024
1 parent 80956df commit 47f7baa
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions config/checkstyle.xml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,13 @@
<!--
<module name="FileLength"/>
-->
<module name="LineLength">
<!-- what is a good max value? -->
<property name="max" value="120"/>
<!-- ignore lines like "$File: //depot/... $" -->
<property name="ignorePattern" value="\$File.*\$"/>
<property name="severity" value="error"/>
</module>

<!-- Checks for whitespace -->
<!-- See http://checkstyle.sf.net/config_whitespace.html -->
Expand Down Expand Up @@ -173,13 +180,6 @@

<!-- Checks for Size Violations. -->
<!-- See http://checkstyle.sf.net/config_sizes.html -->
<module name="LineLength">
<!-- what is a good max value? -->
<property name="max" value="120"/>
<!-- ignore lines like "$File: //depot/... $" -->
<property name="ignorePattern" value="\$File.*\$"/>
<property name="severity" value="error"/>
</module>
<module name="MethodLength">
<property name="tokens" value="METHOD_DEF"/>
<!-- TODO: We should set this value around 40 or 50 -->
Expand Down

0 comments on commit 47f7baa

Please sign in to comment.