А ну и конечно, если человек с чем-то "работал в гитхабе", а вы не работали, с чего вы взяли, что он реально понимает о чем говорит, а не копирует текст из бложиков?

Знаю один реальный инцидент на проде в банке, который возник из-за названия поля в Json которое вводило в заблуждение и означало совсем другое

Знаю один известный проект, который попал в трудную ситуацию из-за увлечения наследованием

Так у вас недостаточно данных, чтобы по коду кандидата, который вы в первый раз видите сказать, правильно он назвал переменную или неправильную

Это понятно. Я знаю десятки проектов, которые умирали из-за переусложнения. Проекта который умер потому что люди называли методы неправильно вот пока не встретил)

Мой опыт показывает, что это не так. В своей репе человек делает так, как сам считает нужным. Его никто не контролирует. И скорее всего он так будет делать, если за ним не следить

Простой пример. Метод называется get что-то ById и в нем запрос к БД в котором селект не только по id. Это я не придумал. Такую поделку видел в реальном проекте и из-за этого были ошибки на проде

Сигнатура это покажет

Вам не кажется, что в своей репе человек вообще не ставит себе цели сделать код читаемым, тестируемым и поддерживаемым, т.к. банально нет ни одного человека, который когда-либо в этот код залезет?

Ну а если залезет, то это уже единицы из опенсорса, у которых в гитхабах реально что-то полезное. О них мы не говорим

Нет. Не покажет. В метод передается только id остальные критерии из констант

А этот список часто шарят между компаниями?

Я могу залить пару проектов на гитхаб. На первом очень хорошая и проработанная архитектура, приятный код, все абстракции реально нужны и используются. Второй проект - нагромождение классов, многие из которых имеют имена типа класснейм1, класснейм2 и т.д. Сущий ад для любого, кто попытается разобраться.
Вот только суть в том, что второй проект я писал уже после первого, и такой код там не случайно. Иногда, решение бизнес задач может потребовать прототипа в максимально сжатые сроки, который потом, при наличии положительного фидбека, будет переписан с нуля уже в адекватном виде. И пишутся подобные проекты через жопу потому что подовляющее большинство из них умрет через 2 недели получив на рынке негативный фидбек.
Как в этом случае судить, не зная контекста?

Я думаю, что если он в своей репе это не ставит, то почему он потом вдруг захочет делать это в рабочей репе. Он скорее всего будет присылать в p/r такой же мусор и потом как одолжение его форматировать, делать читабельным.

Более того, проекты на гитхабе протухают. Тот код, который я писал и заливал 2 года назад - не очень меня представляет сейчас.

Причем может быть как в хорошую сторону (что сейчас пишу лучше), так и в плохую (с тех пор я познакомился с @yegor256 и у меня ООП головного мозга)

Это как смотреть на костыли/говнокод в рабочем проекте и сразу делать вывод о человеке, что он засранец

Без контекста я бы такие диагнозы не ставил. Есть ситуации, когда человек вынужден так писать или там по другому никак (хотя с первого взгляда кажется, что можно лучше)

Я поэтому не люблю тестовые - ты не можешь объяснить свои решения, их обычно смотрят, делают какой-то вывод вне контекста и все

Откуда такой вывод? Зачем мне когда я хочу отдохнуть от всех этих тестовых покрытый, солидов и прочего булшита в совей репке снова всему этому следовать?) Это же мазохизм какой-то

От тестовых и гитхаба можно отталкиваться, но прям такие выводы делать, что человек "скорее всего будет присылать в p/r такой же мусор и потом как одолжение его форматировать, делать читабельным" - не стоит, имхо

Ну, во-первых. Я не хочу спорить. Был вопрос, я на него ответил. Я не претендую на абсолютную истину. Во-вторых - кривые репы можно сделать private. В-третьих MVP тоже нужно уметь делать аккуратно. Это не занимает много времени. Это привычка. Она или есть или нет. Если ты слепил какое-тр быстрое решение для проверки бизнес-идеи оно должно хотя бы подлежать проверке: ту мы идею проверяем или не ту. И потом это решение потом может стать нормальной задачей. И тогда коллегам может понадобится в ней разобраться.

Вы знаете, ситуации бывают разные. На моём опыте была ситуация, когда из-за проблем личного характера  я отказался от предложения, но меня уговорили подписать выход на месяц вперед и там посмотреть по ситуации. И буквально за пару дней до выхода произошёл тот самый неприятный момент, пришлось всё отменить. Мне кажется всё зависит исключительно от ситуации, иногда компании сами предлагают независимо от сложностей принять, а там главное Hrам уже доломать кандидата