Learned something similar on a previous job. Prefix it with "#must", "#should", "#could", "#would".
We've also extended it with some other things that you may want to comment, but are not necessarily actionable. "#wont", "#idea", "#tip", "#question", "#risk", are pretty much self-explanatory.
"#fun": an ugly workaround that could've worked; some funny situation that the change could cause; ...
"#domain-knowledge"/"#debrief": really nice if you are working with people that are more junior, or not that familiar with this codebase in particular. May help other reviewers understand better why the author made certain decisions.
Learned something similar on a previous job. Prefix it with "#must", "#should", "#could", "#would".
We've also extended it with some other things that you may want to comment, but are not necessarily actionable. "#wont", "#idea", "#tip", "#question", "#risk", are pretty much self-explanatory.
"#fun": an ugly workaround that could've worked; some funny situation that the change could cause; ...
"#domain-knowledge"/"#debrief": really nice if you are working with people that are more junior, or not that familiar with this codebase in particular. May help other reviewers understand better why the author made certain decisions.