checkstyle маразмы

Сабина
Уже с Приветом
Posts: 19041
Joined: 11 Jan 2012 09:25
Location: CA

checkstyle маразмы

Post by Сабина »

согласны?

Number 1

Code: Select all

 (indentation) Indentation: try at indentation level 16 not at correct indentation, 12
Number 2:

Code: Select all

(extension) TreeWalker: Got an exception - expecting EOF, found '}'
https://www.youtube.com/watch?v=wOwblaKmyVw
Сабина
Уже с Приветом
Posts: 19041
Joined: 11 Jan 2012 09:25
Location: CA

Re: checkstyle маразмы

Post by Сабина »

Какие все же есть сволочи на белом свете :)!
Китаезы насовали check style ignores и плодят PR-ы пачками с 200+ сheckstyle errors, а я как дура сижу все свои пять github projects вылизываю чтоб не единого warnings. Feeling stupid :angry:
https://www.youtube.com/watch?v=wOwblaKmyVw
User avatar
Uzito
Уже с Приветом
Posts: 8239
Joined: 06 Feb 2002 10:01
Location: NJ, USA

Re: checkstyle маразмы

Post by Uzito »

Во народ страдает. А я-то, дурак-дураком, настроил автоформаттер и о проблеме забыл.
User avatar
АццкоМото
Уже с Приветом
Posts: 15276
Joined: 01 Mar 2007 05:18
Location: VVO->ORD->DFW->SFO->DFW->PDX

Re: checkstyle маразмы

Post by АццкоМото »

Uzito wrote:Во народ страдает. А я-то, дурак-дураком, настроил автоформаттер и о проблеме забыл.
ну, если автоформаттер решает все проблемы, которые ловит чекстайл, то вы - гений-гением :gen1:
Мат на форуме запрещен, блдж!
User avatar
Uzito
Уже с Приветом
Posts: 8239
Joined: 06 Feb 2002 10:01
Location: NJ, USA

Re: checkstyle маразмы

Post by Uzito »

Я сиквели пишу, у меня checkstyle фашизма нету.
Easbayguy
Уже с Приветом
Posts: 10632
Joined: 17 Jul 2003 22:11

Re: checkstyle маразмы

Post by Easbayguy »

Сабина wrote:Какие все же есть сволочи на белом свете :)!
Китаезы насовали check style ignores и плодят PR-ы пачками с 200+ сheckstyle errors, а я как дура сижу все свои пять github projects вылизываю чтоб не единого warnings. Feeling stupid :angry:
Началось :-)
А я сегодня увидел у нас робкие ростки микросервисов, сразу подумал о Сабине!
Пх'нглуи мглв'нафх Ктулху Р'лайх угахнагл фхтагн
Palych
Уже с Приветом
Posts: 13683
Joined: 16 Jan 2001 10:01

Re: checkstyle маразмы

Post by Palych »

"А вот я работал в театре. Капельдинером... Тоже ужас!"
У нас как бы обратная проблема: в некоторых местах check style отключен (не смогли договориться)
Там контрактов такое творят - жуткая жуть...
Я всё думал - как они могут читать код с такой indentation?
Кажется понял: они не читают код. Даже свой.
Сабина
Уже с Приветом
Posts: 19041
Joined: 11 Jan 2012 09:25
Location: CA

Re: checkstyle маразмы

Post by Сабина »

Easbayguy wrote:
Сабина wrote:Какие все же есть сволочи на белом свете :)!
Китаезы насовали check style ignores и плодят PR-ы пачками с 200+ сheckstyle errors, а я как дура сижу все свои пять github projects вылизываю чтоб не единого warnings. Feeling stupid :angry:
Началось :-)
А я сегодня увидел у нас робкие ростки микросервисов, сразу подумал о Сабине!
Да ладно че сразу истерите :) Просто вторую неделю разгребаю жуткий brain fart с каментами /Todo , remove this при том что this используется в 100+ ключевых компонентах, а сделать все надо к release train который каждую пятницу. А тут китаезы радостно включили чекстайл в текущей версии base project, предварительно насовав игнора везде где им надо :) Мои ошибки кстати большинством изза несовместимости maven плагина с Джава 8 синтаксисом, но не хочу в их base Pom лазить, ещё с этим сейчас не хватало возится
https://www.youtube.com/watch?v=wOwblaKmyVw
Сабина
Уже с Приветом
Posts: 19041
Joined: 11 Jan 2012 09:25
Location: CA

Re: checkstyle маразмы

Post by Сабина »

АццкоМото wrote:
Uzito wrote:Во народ страдает. А я-то, дурак-дураком, настроил автоформаттер и о проблеме забыл.
ну, если автоформаттер решает все проблемы, которые ловит чекстайл, то вы - гений-гением :gen1:
Treewalker ошибка это вообще какое то полное хз. С виду похоже на несчастное отсутсвие end of line character , а копнёшь и оказывается что он бедолага неполюбил default method definition в интерфейсе. Великий и могучий чекстайл он ещё и очень загадочный местами :shy:
https://www.youtube.com/watch?v=wOwblaKmyVw

Return to “Вопросы и новости IT”