Roman Ivanov
Roman Ivanov
in same commit is ok.
This is not good, we should reuse existing parser. I can not help on the go, I need big screen time to find solution and guide you. It would be...
still valid. We need to update all of them to start with "Input" for sure. please read https://github.com/checkstyle/checkstyle/issues/6207#issuecomment-523013433 @rnveach , can you share details on folders name ?
We need all sequence of PRs to close this issue. @MANISH-K-07 , please split your big PR in multiple and send them to us. Single review only for such PRs...
Yes, main point is no names on file header. I am ok with proposal.
yes, issue is approved
usage of Eclipse auto formatting is fine, but it should be in non-conflicting with checkstyle rules.
>Nobody should ever reformat code manually to fulfill Checkstyle rules Checkstyle is not only about formatting, but yes we have bunch of spacing related Checks. But we need to find...
Please keep sending PRs with partial updates, reviewing huge PR is not a fun
@Calixte , there are about 4 PRs assigned to you for review , please find time to review them or give us some confirmation that you ok to merge without...