Issue #135: Implemented recipe for EmptyStatement#136
Issue #135: Implemented recipe for EmptyStatement#136tannmayygupta wants to merge 6 commits intocheckstyle:mainfrom
Conversation
|
These tests should extend |
|
yess sir, i see that other recipe implementations extends AbstractRecipeTestSupport will extend and raise another PR sir. |
…or automatic empty statement removal - Remove standalone empty statements (double semicolons, standalone semicolons) - Replace empty control flow bodies (if/while/for/do-while) with empty blocks - Add comprehensive test coverage with 16 test scenarios - Support both XML and SARIF report formats
|
Sir, implemented EmptyStatement recipe that extends abstractRecipeTest as it was done for other recipe. |
…o add copyright header
|
@tannmayygupta please fix ci failures and squash all commits into one |
|
@rdiachenko Yes sir, working on it, will fix and squash into one commit. |
|
@Anmol202005 @romani @rdiachenko Sir I've fixed the CI failure and tried to squash in one but the (git rebase -head~3). Please review and let me know if any changes are needed. |
|
Sorry Sir, Didn't expected that it will fail. I read all the previous CI Failure error and tried to fixed them. I will start working on present CI failure. |
|
@Anmol202005 @romani @rdiachenko Sir I've understand and fixed the CI failure. Please review and let me know if any changes are needed. |
|
@tannmayygupta pitest mutations introduced by your code must be fixed, but not suppressed as you did. Same for checkstyle suppressions. Please make sure this PR doesn't introduce any new suppressions. |
|
@rdiachenko Understood Sir. I'll fix the mutations by improving the code logic instead of using suppressions. working on it. |
|
Sir, tried fixing EmptyStatement mutation without suppressions. Please review sir and let me know if any changes are needed. |
|
Always single commit in PR, please . |
|
Sorry sir, will do it in one commit. |
4 participants
Description
Implements OpenRewrite recipe for Checkstyle's EmptyStatement check.
This recipe automatically detects and removes empty statements (standalone semicolons)
that serve no purpose in the code, covering multiple scenarios including control structures.
About Me
Looking forward to contributing more to Checkstyle! 🚀