I feel that the guidance is where most people fall. There is an opinion that the code should explain itself, so no need for code comments or review comments, but I don’t agree with it.
I use the comments also to point the reviewer to the meatier parts, and where I want them to pay more attention.
Great article Akos!
In my view, PRs should also follow the single-responsibility principle. At the very least, they will be easier to roll back in case of any issues.
This is a very useful and actionable guide. Thank you for sharing!
I feel that the guidance is where most people fall. There is an opinion that the code should explain itself, so no need for code comments or review comments, but I don’t agree with it.
I use the comments also to point the reviewer to the meatier parts, and where I want them to pay more attention.
Awesome tips, Akos! I try to get my mentees to do these ones too :D.
Thanks for the shout-out too