Когда человек делает тестовое задание на 2к линий и на 22 файла (на Go), то у меня возникает паника: а на какое безумство готов пойти этот человек в команде? Не, конечно же хочется написать хорошее решение, потом его сделать публичным и получить 2й прометеус, но реальность чуть другая и тестовое задание не более чем маркер - говорить дальше или нет (даже разговор потом это не 100% гарантия хорошего кандидата, кэп). А вот душнота в тестовом - неприятная вещь. Ты вроде читаешь и пытаешься понять, как думал человек, а в итоге смотришь на его/её логгер или 3 варианта решения для стореджа. Ну приятно, но смысла мало, если бы я увидел комент "вот тут на каналах, но можно на мютексе" - я бы был более рад. Но и 150 строк без объяснения в одном файле это другая сторона - анти-душность. Вроде и правильно написано, но непонятно, а что человек может еще? или хотя бы может о чем-то подумать. Наверное в виде совета сказал бы такое - ревьюверы не хотят делать ревью, ну вот прям такое супервдумчивое с 2мя созвонами по 1 фции, ну как с рабочим репо, но при этом хочется видеть о чем думает и о чем может думать кандидат. Может и не надо писать много коментов в коде, но если в README написать TODO чего не хватает или что хочется потрогать (фаззинг/jepsen/etc) то лучше сказать, мне это не раз помогало найти крутых кандидатов.