Roman Ivanov
Roman Ivanov
Ok. So you are ok to have files named as "Input{CheckName}CheckXxxx.java" ? To follow name pattern of test IT inputs .
Sorry, copied wrong pattern. Ok, IT pattern should be `Input{CheckName}Xxxx.java`. I recommend to add such pattern in validation and allow two patterns (with input prefix and existing ) while work...
@mincong-h , do you have time to continue ?
@Binita-tech , This collection should be empty https://github.com/checkstyle/checkstyle/blob/4e820940caf04662200a51c13f7befd7135cbbe7/src/test/java/com/puppycrawl/tools/checkstyle/internal/XpathRegressionTest.java#L99-L102 Please only one Check update by one PR
List in github might be outdated Actual list of what is not done yet is https://github.com/checkstyle/checkstyle/blob/628a7643aeaa6736bec4c7c38910e5c0ab26aa4f/src/test/java/com/puppycrawl/tools/checkstyle/internal/XpathRegressionTest.java#L92 See commits referencing this issue as guidance on what to update and how
@pbludov , I am changing it to bug, as there is no new token that user choose to select.
I do not remember why we did two config per project, one for master one for PR, may be some old TC limitations.
>Do we want to investigate consolidation of master and PR configs It makes sense to ask support, badges might be improved in last 5 years so such problem might be...
@nrmancuso , please recheck issue completion. I think it is done.
approved. it would be good to see html report(s) where ALL is activated to see visually scope of work. REpost should be on github.io to let mentors review it and...