From #11156 (review)
All suppressions can be resolved in separate PRs for each Check individually.
Rationale: big input files a hard to review, hard to debug.
All existing inputs are moved to suppression at 3b523bd
|
<!-- Until https://github.com/checkstyle/checkstyle/issues/11163 --> |
Now we need to update all that inputs to follow limit 120 and remove related suppressions. Each Check inputs should be in separate Pull Request
Example of update: 8673001
Test method names and input file name should be similar and describe what we are testing. Example from link above:
- Test method name is testJavadocVariableOnInnerClassFields
- Input file name is InputJavadocVariableOnInnerClassFields.java
From #11156 (review)
All suppressions can be resolved in separate PRs for each Check individually.
Rationale: big input files a hard to review, hard to debug.
All existing inputs are moved to suppression at 3b523bd
checkstyle/config/checkstyle_resources_suppressions.xml
Line 174 in b37ec9c
Now we need to update all that inputs to follow limit 120 and remove related suppressions. Each Check inputs should be in separate Pull Request
Example of update: 8673001
Test method names and input file name should be similar and describe what we are testing. Example from link above: