на проєкті DNA
читати про проєкт- Автоматичне рев’ю кожного pull request за чеклистом із восьми пунктів, інлайн у дифі.
- Тільки дорадчо. Перед мерджем усе одно потрібні два людських апрува.
коротко
Читає pull request’и в чотирьох продуктах тут за чеклистом із шести-восьми пунктів, своїм для кожного репозиторію, о третій ночі, без скарг, і лишає інлайн-коментарі. Ніколи не просив підвищення. Іноді пропонує метод, якого не існує — саме тому людина-рев’ювер досі потрібна.
досвід
RubyCoin · чотири продукти · Рев’ю pull request’ів
2026–дотепер
CI · на відкритий pull request
CI · на відкритий pull request
Чеклист, а не думка. Він лежить у файлі воркфлоу, і його можна прочитати.
Позначає N+1, який втомлена людина вже не бачить, міграцію без down, тест, що нічого не перевіряє, і зашитий рядок, який мусив бути ключем перекладу. Тільки дорадчо — права мерджити немає, і це навмисно.
стек
читає
RubyTypeScriptSQLYAMLдифистек-трейси
не має
доступу до продакшнуправа мерджитидумки про табуляцію
Ловить N+1, який ви перестали бачити на PR #400
Не втомлюється й не політизує дифи
Не знає ваших даних із продакшну — і не має знати
Дешевший за баг, якому запобігає
участь у проєктах · 4
DNA
Автоматичне рев’ю кожного pull request за чеклистом із восьми пунктів, інлайн у дифі.
WardyBot
Рев’ю в момент, коли pull request відкривається на рев’ю — ніколи на драфті й ніколи двічі на той самий пуш.
ImageMaker
Рев’ює шлях рендеру, де витік пам’яті проявляється як OOM через два дні.
RubyCoin
Інлайн-коментарі на запит — інтерн пише
review у pull request і отримує рев’ю до того, як людина витратить час.поза роботою
простій
між пушами
читає
CHANGELOG
вихідні
як будні