Comments 6
Цыгане шумною толпою
Учили прогеров гадать )))
В ADR есть четыре составляющие:
Prologue (Summary)
Discussion (Context)
Solution (Decision)
Consequences (Results)
В описанном PDR нет Discussion, то есть опций и контекста, из которых делался выбор.
Спасибо за статью. Интересный подход.
Правда пример с код ревью не очень зашёл. Код ревью предназначен не только для поиска ошибок, в большинстве случаев достаточно сложно найти ошибку просто смотря на код. Для меня это скорее способ обучения и менторства. Например, ты видишь, что кто-то добавил новый компонент, который ты сможешь заиспользовать в будущем или наоборот, кто-то пишет компонент с нуля в то время, когда он уже существует и ты можешь рассказать об этом.
Ну и зачастую находятся более высокоуровневые ошибки, когда, например, задача решена, код работает, но архитектурно задачу можно было бы решить гораздо проще. А возможно данное решение хоть и работает, но не вписывается в концепцию текущей реализации и его будет сложно поддерживать.
Конечно, многие проблемы можно отловить ещё до написания кода, обсудив реализацию, но иногда вообще непонятно как решать задачу и это не удастся сделать.
спасибо за отзыв!
Про использование код-ревью для обучения я тоже писал, если интересно, вот: https://hackernoon.com/code-review-its-bad-expensive-and-ineffective-in-most-cases
И Фил Дельгадо делал замечательный доклад https://www.youtube.com/watch?v=4Y0XJXRZv6k
Process Decision Record простой инструмент постепенной рационализации процессов