Mauryan Kansara
Mauryan Kansara
checker gave an exception at: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/7007ab5_2025074153/reports/diff/apache-struts/index.html#A17 @mohitsatr can you investigate it? Exception was thrown at following line: https://github.com/checkstyle/checkstyle/blob/7007ab5d4de1f18bc51c18f7bbdf383e9cdac9af/src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/TextBlockGoogleStyleFormattingCheck.java#L198 But it was for for a multi-line comment, so it maybe not...
False-negative. Opening quotes are not violated: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/7007ab5_2025074153/reports/diff/camunda/index.html#A421 Case: ```java private static String createMigrationRejectionDueConcurrentModificationReason( final long processInstanceKey) { return """ Expected to migrate process instance '%d' \ but a concurrent command...
Found similar case again, in Checkstyle codebase: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/7007ab5_2025074153/reports/diff/checkstyle/index.html#A3 57th line is not marked as violation. Another one: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/7007ab5_2025074153/reports/diff/checkstyle/index.html#A101 41st line is not violated.
False positive: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/7007ab5_2025074153/reports/diff/spring-integration/index.html#A24
Exception at: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/44e98b8_2025131835/reports/diff/checkstyle/index.html#A208
https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/44e98b8_2025131835/reports/diff/spring-integration/index.html#A95 It is showing violation for line 243, please ensure that the ending `"""` is preceded with tabs, only then it should throw violation for it otherwise it is a...
Please add: `Part of https://github.com/checkstyle/checkstyle/issues/17778, subproblem 1` just to keep track of issues more clearly.
please open an issue about this on their Github.
> Since Google config doesn’t use MissingOverrideCheck, should these comments remain as they are? No, the issue #17561 is related to Google Style guide, indicating a gap in our implementation....
You can squash the commits into one and force push it.