Roman Ivanov
Roman Ivanov
@nick-mancuso , do we need this mega issue ? I think it is better to create more precise issue with more clear scope and target.
Yes, select one, do comment "I am on ..." and send PR. Please keep in mind, it is better to resolve first gsoc, second gsoc , before doing this "third".
>EmptyLineSeparatorCheckTest Will be covered at https://github.com/checkstyle/checkstyle/pull/11408 @Kevin222004, fyi.
Yes it should be updated to use execute method
loading of config from Input file very helpful when we do violations in such Input file, so we review config and violations in same place. In case of test that...
Remove your change and run again it should pass, after that add changes gradually to get understanding what cause this problem.
Scroll up to see more detailed error message, input file might miss/extra some '//violation ' messages or some other problem.
Keep digging, do debug of tests to get more insight. Look at other commits that are done by others to see how they did same updates. Please show your ability...
Please read https://github.com/checkstyle/checkstyle/issues/12011#issuecomment-1207446712 Please always show testing output. Generate release notes for few past releases. Script need to updated to get by API release that is previous before required.
your tweet is not correct. tweet should be like https://twitter.com/checkstyle_java/status/1508102605314043906?s=20&t=aK-niouv-PuFHvT6zyfurQ