Time for a meeting
If a discussion thread on a pull request exceeds two or three replies or if a pull request is idle with no progress being made on the code review, then it is time to do the code review synchronously and go over the pull request in a meeting.
Similarly if you are rushed on time and need to land a pull request by a certain deadline then sending the pull request followed directly by a code review meeting will help the pull request to be reviewed and approved on time.
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.