Keep it friendly
When code reviewing, put your comments in forms of suggestions and questions.
Instead of saying move this logic to a new class say looks like this class is doing a lot. Should this logic be in a new class?
This keeps the review friendly and constructive.
Software Engineering from the Frontlines Course on Maven
If you liked this article, I will be teaching a “Software Engineering from the Frontlines” course on Maven where I will teach hard-learned lessons I acquired developing large-scale products at companies such as Uber, Airbnb, and Microsoft.
Thanks for reading Software Engineering Tidbits! Subscribe for free to receive new posts and support my work.