Дэвид столкнулся с запутанным pull request, содержащим обширные комментарии, касающиеся, казалось бы, простого изменения: добавления одного параметра в JavaScript-функцию. Изначальная функция уже имела семь параметров, что потенциально могло указывать на возможность рефакторинга с использованием объекта. Первая ревизия представила условное выражение, связанное с новым параметром notArg8. Если notArg8 было истинным, код передавал arg8: !notArg8 в контекст. Автор не смог понять этот сложный подход, поскольку можно было просто добавить параметр напрямую. Дэвид оставил комментарий о плохом коде, но исходный разработчик перестал отвечать. Младший разработчик одобрил запутанное изменение кода, но оно так и не было объединено. Позже другой разработчик упростил решение, просто добавив arg8 и передав его в context. Упрощенный код работал, проходил тесты и удовлетворил всех. Намерение исходного разработчика и обоснование сложного условия остаются загадкой. Концовка намекает, что видение исходного разработчика осталось нереализованным.
notArg8. ЕслиnotArg8было истинным, код передавалarg8: !notArg8в контекст. Автор не смог понять этот сложный подход, поскольку можно было просто добавить параметр напрямую. Дэвид оставил комментарий о плохом коде, но исходный разработчик перестал отвечать. Младший разработчик одобрил запутанное изменение кода, но оно так и не было объединено. Позже другой разработчик упростил решение, просто добавивarg8и передав его вcontext. Упрощенный код работал, проходил тесты и удовлетворил всех. Намерение исходного разработчика и обоснование сложного условия остаются загадкой. Концовка намекает, что видение исходного разработчика осталось нереализованным.