Roman Ivanov
Roman Ivanov
Merged at https://github.com/checkstyle/checkstyle/pull/16946
Sorry, workflow is broken by recent update. Please use old workflow trigger.
@mahfouz72 , please use `OR as alternate` from https://github.com/checkstyle/test-configs/tree/main/WhitespaceAround/all-examples-in-one#all-examples-in-one-configs Please share feedback on how to organize Readme to let be more intuitive.
Github, generate report for configs in PR description
Github, generate report for WhitespaceAround/all-examples-in-one
GitHub, generate report by config from src/test/resources-noncompilable/com/puppycrawl/tools/checkstyle/checks/whitespace/whitespacearound/InputWhitespaceAroundSwitchCasesParensWithAllowEmptySwitchBlockStatements.java
GitHub, generate report by config from src/test/resources-noncompilable/com/puppycrawl/tools/checkstyle/checks/whitespace/whitespacearound/InputWhitespaceAroundSwitchCasesParensWithAllowEmptySwitchBlockStatements.java
@xenoterracide , do you specifically care about Class, or you think it should apply to types (interfaces, records) in same time or we need extra property to define target. Or...
ok `countInnerClass` --> `countInnerType` as there are interfaces/records/classes. we should skip `countEmpty` for first version of this Check, I do not think anybody care about this in 21th century. countInnerType...
@xenoterracide , do you agree with my proposal to change design ? if yes, we will need to create new issue with clear description of design, final version.