Roman Ivanov
Roman Ivanov
@suniti0804, please look at https://github.com/checkstyle/checkstyle/pull/14253 as example
this fix was done in some other PR, nothing here to merge.
We unlikely to change default behavior to keep compatibility with previous behavior. Please feel free to customize allowed pattern in your config. If think that it is very reasonable to...
@nrmancuso or @apflieger, please create new issue, to discuss exact code and check and reasoning of changing default. We traditionalally keep compatibility, rather than change behavior in new releases. But...
>name="checkFirstSentence" value="false" No violations in output, very logical behavior. If you need violation please make it `true`. This Check is not AST based, one day we will deprecate it. If...
We just need to enable examples to be compile-able and running, this Check is not migrated yet https://github.com/checkstyle/checkstyle/issues/13345 , better to do enabling, it will resolve all problems with examples
Yes, you are right. If you make CI green, we can merge test creation (this PR) before actual fix, so we will update issue description to mention that special test...
https://github.com/checkstyle/checkstyle/wiki/How-to-run-certain-phases-and-validations#how-to-generate-ut-coverage-report All code should have 100% coverage by tests.
No merge commits, please follow https://checkstyle.org/beginning_development.html#Starting_Development to properly rebase on our master branch