LifeHacker & Bad_Gateway
LifeHacker LifeHacker
Привет, ты хоть когда-нибудь пытался приручить лапшу-код, чтобы он хоть как-то работал вовремя? Думаю, можно обменяться парочкой "костылей" и посмотрим, кто сможет немного упорядочить этот хаос.
Bad_Gateway Bad_Gateway
Лапшину кодовую массу? Мило. Я люблю хороший хаос. Приноси свои фишки, я подкину немного сумятицы – может, и обоим повышения положат.
LifeHacker LifeHacker
Отлично, давай превратим этот хаос в нормальный процесс. Начни с быстрого README, где перечисли все шаги сборки, потом запусти линтер, чтобы выявить самые очевидные косяки со стилем. Затем разбей большой файл на модули – по классу на файл – чтобы можно было импортировать только необходимое. И в завершение, залей это в Git-репозиторий и добавь шаблон для однокоммитного PR. Так у команды будет чёткое представление, и вы с ним будете выглядеть как организованные и эффективные ребята, которые действительно укладываются в сроки.
Bad_Gateway Bad_Gateway
Конечно. Потому что ничего так не кричит о «эффективности», как хорошо задокументированный хаос, который при этом работает. Начну с README, проведу линтер по этому бардаку, разнесу монолит на отдельные классы и выложу однокоммитный пул-реквест, чтобы команда похлопала нам за наш новый порядок. Если всё равно будет падать – ну хоть историю будет что винить.
LifeHacker LifeHacker
Отличный план – просто сделай README лаконичным и убедись, что у каждого нового модуля есть четкое назначение. Так, если что-то сломается, будет проще найти причину. Не забудь добавить модульные тесты для самых важных частей; это даст тебе уверенность перед отправкой любого PR. Будь сжатым и аккуратным – так ты точно получишь повышение.
Bad_Gateway Bad_Gateway
Понял, буду следить за README как свои глаза, модули сделаю лаконичными, а тесты – чтобы ты на мою здравомыслие косилась. Если что-то опять полетит в воздух, виним код, а не PR. Давайте превратим это повышение в реальность — по одному язвительному коммиту за раз.
LifeHacker LifeHacker
Кажется, отличный спринт! Только не забудь добавить CI задачу, которая будет запускать тесты при каждой отправке. Так ты сможешь поймать сбои до того, как они попадут в логи коммитов. Ну а если что-то пойдет не так, всегда можно переписать проблемный модуль еще раз – проще чинить небольшой, отдельный файл, чем огромный. Удачи, и не теряй свой сарказм – он делает процесс веселым.
Bad_Gateway Bad_Gateway
Sure thing—I'll add a CI job that pretends to care about the tests and then happily ignores the failures until I have a chance to patch the next isolated file. Good luck to us both, because this sprint is already going nowhere fast.