Roman Ivanov
Roman Ivanov
yes, google style doesnot use special domains https://github.com/checkstyle/checkstyle/blob/9017ae84cd019a0d3b84b15212d9c5c2f8edcaea/src/main/resources/google_checks.xml#L321-L326
Yes, `Diff Regression patch config: {{URI to patch_config.xml}}` should be used to let on patched execution use config with new property.
too much is changed from PR start time, lets abandon this PR for now , we can reopen it after we migrate to new AST for javadoc
@ZiadAbdElFatah, please review this response. Please close issue if it is not actual anymore.
better to ignore order
fix is merged
https://checkstyle.sourceforge.io/filefilters/beforeexecutionexclusionfilefilter.html#Example4-config Whole reason of this Filter exists it to exclude module-info.java `module-info.java` is not java actually, it is separate syntax (special language), and we do not have Parser for this...
We need to create grammar for this module-info.java files. as this is not very frequent file in real life, we can make grammar for this syntax and embed it certain...
> `` we should demand space after "//", we do this alredy in our codebase: https://github.com/checkstyle/checkstyle/blob/dfec36a68198f335475976c47ca7c4f2dd553603/config/checkstyle_checks.xml#L639-L644 it will resolve not all cases, but most of them, unresolved is leading space...
@mohitsatr , please create separate issues for each case/point in this issue, each of them will be handled separately it is not same fix for all of them. Please use...