Roman Ivanov
Roman Ivanov
I closing this issue, we will open new issue is something is required to be improved.
will be handled at scope of https://github.com/checkstyle/checkstyle/issues/13672
will be handled at https://github.com/checkstyle/checkstyle/issues/13672
We should not extend this Check. allowEmptyLineBeforeReturn vs `violation 'return' should be separated from previous line.` It doesn't make sense, and conflicts from naming perspective. Target of EmptyLineSeparator are big...
will be handled by https://github.com/checkstyle/checkstyle/issues/13672
Take one module from https://github.com/checkstyle/checkstyle/issues/6207#issuecomment-521414318 list and do update for it
All files under https://github.com/checkstyle/checkstyle/blob/master/config/checkstyle_resources_suppressions.xml#L178 Should be split in smaller files and removed from suppression
@Shubhra0310, if you want to help us to resolve issue, just do comment "I am on it" and send Pull Request. This issue is big , so put a comment...
@nitish799 , please choose any Check under suppression and please send PR for single Check. you can see above lnks to bunch of example commits and PRs to see how...
@suniti0804 , please reduce list by grouping by Check name. each PR should fix whole Check Inputs.