Roman Ivanov
Roman Ivanov
to make it Input based we should update all our test suite code to expect stripped file path ``` --- a/src/test/java/com/puppycrawl/tools/checkstyle/AbstractModuleTestSupport.java +++ b/src/test/java/com/puppycrawl/tools/checkstyle/AbstractModuleTestSupport.java @@ -575,7 +575,9 @@ public abstract class...
comment it added.
@MohanadKh03, looks like you passed all "good" issues. Do you need suggestion on next or what to choose yourself? Now you are fully onboarded and any issue unlikely be a...
Please put something in super, like `super(1+3, 4*6)`
I am removing "approval" label until we agree on it. we can update examples to show user how to violate names is context where it is confusing - https://github.com/checkstyle/checkstyle/pull/15486#discussion_r1713058911
We need to rethink this Check, something wrong in trend to forbid general words. `record` is good word and used a lot, especially in DB related applications
https://github.com/checkstyle/checkstyle/issues/15499
rebased in web ui
we need to make sure this update is working, so please build and deploy it to your dockerhub, and we will test it in checkstyle repo.
I see you here https://github.com/checkstyle/checkstyle/pull/17235 , you are on right path