Skip to content

Commit

Permalink
Add text summary of the review process #32
Browse files Browse the repository at this point in the history
  • Loading branch information
pintergreg committed Oct 31, 2024
1 parent 40979af commit f1e518c
Showing 1 changed file with 14 additions and 1 deletion.
15 changes: 14 additions & 1 deletion lectures/14_code_review.md
Original file line number Diff line number Diff line change
Expand Up @@ -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?
:::::::::
::::::::::::


#
Expand Down

0 comments on commit f1e518c

Please sign in to comment.