From f1e518cfb67a85ee63d84957e74c97aa8ade2f41 Mon Sep 17 00:00:00 2001 From: pintergreg Date: Thu, 31 Oct 2024 18:17:18 +0100 Subject: [PATCH] Add text summary of the review process #32 --- lectures/14_code_review.md | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/lectures/14_code_review.md b/lectures/14_code_review.md index 8334db6..2950a5a 100644 --- a/lectures/14_code_review.md +++ b/lectures/14_code_review.md @@ -27,7 +27,20 @@ Sometimes it can be somewhat formal. # a review process -![](figures/proc3.png){width=550} +:::::::::::: {.columns .column-gapless} +::::::::: {.column width="60%"} +![](figures/proc3.png){width=500} + +::::::::: +::::::::: {.column width="40%" .mt-5 .text-smaller} +- is it understandable (clear) +- is it clean (no code smells) +- does it match the task? +- does it fulfill the task? + - every [DoD]{.tooltip title="Definition of Done"} point covered? +- is it possible to improve? +::::::::: +:::::::::::: #