Автор скрипта не проверяет его в одиночку. В задаче он фиксирует цель, список затрагиваемых аккаунтов и сущностей, ожидаемый результат и способ отката. Без этого ревьюер видит строки кода, но не понимает границы изменений.
На ревью проверяем четыре вещи:
— нет ли ключей и токенов внутри кода;
— ограничена ли выборка нужными кампаниями;
— учтены ли лимиты, часовые пояса и валюты;
— пишутся ли ошибки и результаты выполнения в лог.
Отдельный пункт — повторный запуск. Скрипт не должен создавать дубли, повторно менять уже обработанные объявления или падать на пустой выборке. Если операция необратима, сначала добавляем режим без внесения изменений: он показывает, что именно попадёт под обработку.
Перед рабочим запуском прогоняем код на тестовом аккаунте или минимальном наборе кампаний. После запуска сверяем логи, количество обработанных сущностей и фактические изменения в интерфейсе.
Правило команды: автор отвечает за решение, ревьюер — за независимую проверку, а запуск начинается только тогда, когда понятны последствия и откат.
Контекст на совещании
@context_ops
Как мы ревьюим код объявлений перед запуском и не ловим ошибки на рабочем аккаунте
Этот пост опубликован в Telegram-канале Контекст на совещании. Подписаться можно по ссылке: @context_ops.