I started to do code reviews by pulling down the branch on my computer. For me it's easier to reason about the changes. What do you think?
BTW I've been down the deep end of maximizing review quality and mistake detection. Ran every tool imaginable in CI + super thorough reviews
-
-
100% mutant, flay, flog, reek, rubocop, rubocop-rspec, as well as the type of reviews we are talking about where reviewer plays with code.
-
So if you are maximizing quality of any kind then
. If code being reviewed is likely to churn as the author learns what they want then
End of conversation
New conversation -
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.
It’s not a matter of trust, but to increase the chances of spotting mistakes. This is good for the entire team.