1. отсутствие комментариев
2. отсутствие сборки кода
3. отсутствие форматирование
4. не используют линтеры
5. портянки кода
6. бесчисленные fmt.Println("тут что-то")
7. panic всюду... ну прямо совсем... чуть ли не любую ошибку так обрабатывают
8. не делают нормального продового структурирования на пакеты
9. не пишут документацию от слова совсем
10. часто делают не то что надо
11. если пишут тесты то только позитивные сценарии, часто негативных сценариев почти не найти и это проблема
12. часто код скушный, не интересный и когда все эти пункты на него наслаиваются впечатление портится
Про 1 пункт. Я вижу какие-то разные мнения. Часто встречаю, что "ууу, код не закомментирован - плохой". Но в чистом коде Р. Мартин наоборот грязью поливал комментарии.