rnveach
rnveach
> why after such massive fixes Each check we fixed so far had only around 1 mutation to fix. The pitest reports used in those repos were already the latest...
> count amount of survivals Yes this should be possible. I was also thinking of finding a way to show number of survivals for each line. Then we would know...
Mutator is not completely finished until suppression list is resolved. https://github.com/checkstyle/checkstyle/commit/3ef61de60c1ae93cf0d9a06c56ad84cfedeefa88#diff-4e0a49a8be36cb3b594f01212755dcf16d2f4fc701ded58ee7404f7180ed958fR148 If it won't be done as part of this issue, then we need a new issue to cover it.
> coverage for FullIdent is in https://github.com/checkstyle/checkstyle/pull/11231 , we would have to merge that first or add suppressions here to make CI green.
```` [INFO] --- maven-pmd-plugin:3.17.0:check (default) @ checkstyle --- [INFO] PMD version: 6.48.0 [INFO] PMD Failure: com.puppycrawl.tools.checkstyle.Checker:64 Rule:TooManyMethods Priority:3 This class has too many methods, consider refactoring it.. [INFO] PMD Failure:...
> This rule should be ignored. @pbludov Just curious why we can't configure PMD to act the same as our check? Usually we try to do that as I have...
Old Idea: Our current module identification process is very strict. This PR is to allow that to be loosened so 3rd parties can create their own sub-modules that our current...
CI is passing here. https://github.com/checkstyle/checkstyle/actions/runs/3262167551#artifacts No reported checker differences.
Your limited in where this logic can go as PackageObjectFactory#createModule is called by the CLI (roots), checker (file sets), and treewalker (abstract checks) The best way to make this non-static...