Hi all,

I noticed that throughout the codebase, when method arguments wrap to a new
line, there are cases where indentation is 4 and other cases align the
wrapped line to the previous line of argument.

The latter is caused by intelliJ settings of "Align when multiline"
enabled. This won't be flagged by checkstyle due to not setting
*forceStrictCondition* to *true*

https://checkstyle.sourceforge.io/config_misc.html#Indentation_Properties

I'm suggesting setting this to true to avoid the discrepancy and redundant
diffs in PR caused by individual IDE settings. People who have set "Align
when multiline" will need to disable it to pass the checkstyle validation.

WDYT?

Best,
Raymond

Reply via email to