Отступ при форматировании флюентов #2812
Conversation
|
Kudos, SonarCloud Quality Gate passed! |
|
Не хватает опции на отключение :) Вдруг я не хочу чтобы мне автоматом так форматировало |
а есть варианты, когда такое поведение не нужно? |
Код стайл команды, например |
| @@ -246,6 +246,11 @@ public String getNewText( | |||
| newTextBuilder.append(StringUtils.repeat("\n" + currentIndentation, token.getLine() - lastLine - 1)); | |||
| } | |||
|
|
|||
| if (needNewLine && tokenType == BSLLexer.DOT && additionalIndentLevel < 0) { | |||
There was a problem hiding this comment.
< 0 ?
Кажется, ты нашел какой-то баг.
There was a problem hiding this comment.
ятупо скопипастил снизу=)
There was a problem hiding this comment.
There was a problem hiding this comment.
возможно баг там был изначально! :D
в любом случае, тесты есть, зеленые. опцию в конфиг... ну не знаю. можно, но не булево тогда, а как во взрослых редакторах типа идеи - размер отступа при чейнинге задается отдельный. базовый для джавы, например, это 4 для обычного отступа и 8 для чейнинга.
думал, может в спеке есть новый параметр, а мы проглядели, но нет, глянул спеку, ничего такого там нет сейчас :(
There was a problem hiding this comment.
настройка базового отступа есть в протоколе. думаешь, надо добавить в конфиг? типа расширить базовую конфигурацию?
There was a problem hiding this comment.
там просто несовсем понятно, как разрешать коллизии. хотя при явном вызове форматтера есть явные formatiing options. их можно брать в приоритет.
There was a problem hiding this comment.
настройка базового отступа есть в протоколе. думаешь, надо добавить в конфиг? типа расширить базовую конфигурацию?
Угу, расширить
There was a problem hiding this comment.
Настройка отступа точно нужна)
Пока не найдете мне живого 1сника, которому нужно другое поведение, не поверю
Описание
Отступ при форматировании флюентов
Связанные задачи
Closes #2811
Чеклист
Общие
gradlew precommit)Для диагностик
Дополнительно