Mauryan Kansara
Mauryan Kansara
We got our answer: https://github.com/google/styleguide/issues/941#issuecomment-3320786633 All of the 3 subproblems mentioned in the issue are valid.
Re opening the issue. We're still not following this rule properly, the PR helped us to support it only partially. I'll open sub-issues when I'll get on my laptop.
Yeah. New sub-issues of this issue.
> It does not validate cases where non-whitespace characters appear before the `*/`, even though this situation is explicitly described in the check’s documentation. As a result, the following example...
Please add: ``` Part of [17778](https://github.com/checkstyle/checkstyle/issues/17778), subproblem 3 ``` to keep track of issue clearly.
> If we can add support for validating the alignment of the closing `*/`, it would improve compliance with the Google Javadoc style. Yes, we can implement this. Currently, the...
I think Indentation checking should be easy too. We first check if both quotes are at the start of line or not. And then check if both quotes are vertically...
> we will first find the indentation of a opening quotes and indentation of it's parent element (anything that's one line above it). if the difference between the two values...
@mohitsatr please start generating regression reports.
@mohitsatr ping