You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I thought this fixer is great fit in combination with PSR-2's recommended 120 line length limit.
However, it looks like line_break_between_method_arguments is trimming the line from whitespaces and only then starts to calculate if content is longer than max-length.
This makes behavior quite non-compatible and non-predictable, because max-length is depending on indentation of current line. If I set it to 120 lines with intention to stick with PSR-2 limit, this will not work for intended code like so
^ fixer changes above because even though code after fixing is longer than 120 lines, without whitespaces it's not
Attributes can be nested infinite times, which will push the limit towards PSR's 120 line length limit, but max-length option does not care about that
The text was updated successfully, but these errors were encountered:
ostrolucky
changed the title
line_break_between_method_arguments should from beginning of line, including whitespaces
line_break_between_method_arguments should count from beginning of line, including whitespaces
Nov 20, 2023
I thought this fixer is great fit in combination with PSR-2's recommended 120 line length limit.
However, it looks like line_break_between_method_arguments is trimming the line from whitespaces and only then starts to calculate if content is longer than
max-length
.This makes behavior quite non-compatible and non-predictable, because max-length is depending on indentation of current line. If I set it to 120 lines with intention to stick with PSR-2 limit, this will not work for intended code like so
^ fixer changes above because even though code after fixing is longer than 120 lines, without whitespaces it's not
Attributes can be nested infinite times, which will push the limit towards PSR's 120 line length limit, but
max-length
option does not care about thatThe text was updated successfully, but these errors were encountered: