Follow up of #16760
In the above mentioned PR, validation is applied to all the input files to mention all the default properties and to compulsorily use "default" tag for default properties and only finalparameters package is updated.
TODO:
Update Input Files such that all default properties should be mentioned and "default" tag must be used for default properties.
After updating all the Input files of a module remove the module name from the SUPPRESSED_MODULES set in the inlineConfigParser.
Note: Update a Single Package in a Single PR .
Modules with pending input file updates:
Follow suppression list at
|
private static final Set<String> SUPPRESSED_CHECKS = Set.of( |
|
"com.puppycrawl.tools.checkstyle.checks.coding.MultipleStringLiteralsCheck", |
|
"com.puppycrawl.tools.checkstyle.checks.design.DesignForExtensionCheck", |
Choose one Check and send PR for it
Good example of expected update:
https://github.com/checkstyle/checkstyle/pull/17218/files
https://github.com/checkstyle/checkstyle/pull/17477/files
Follow up of #16760
In the above mentioned PR, validation is applied to all the input files to mention all the default properties and to compulsorily use "default" tag for default properties and only
finalparameterspackage is updated.TODO:
Update Input Files such that all default properties should be mentioned and
"default"tag must be used for default properties.After updating all the Input files of a module remove the module name from the
SUPPRESSED_MODULESset in theinlineConfigParser.Note: Update a Single Package in a Single PR .
Modules with pending input file updates:
Follow suppression list at
checkstyle/src/test/java/com/puppycrawl/tools/checkstyle/bdd/InlineConfigParser.java
Lines 238 to 240 in 180b50d
Choose one Check and send PR for it
Good example of expected update:
https://github.com/checkstyle/checkstyle/pull/17218/files
https://github.com/checkstyle/checkstyle/pull/17477/files