IME a 5 minute conversation (phone, Zoom/Teams/Hangout/...) can replace many, many words of comments and dramatically reduce the latency of the whole PR process.
Comments
Log in with your Bluesky account to leave a comment
When I'm working with teams I'll ask a group what "Mary had a little lamb" means. Most will say, "It's from a nursery rhyme", which is true. But, I point out, it could also mean that Mary was eating lamb & wasn't very hungry. Or Mary is a sheep and gave birth. So...
the trick is determining which version of "Mary had a little lamb" we mean. Yes, you can do that in comments but a conversation is much faster and more clear.
Of course, that doesn't take into consideration different timezone, working hours, etc. So...
Comments
Of course, that doesn't take into consideration different timezone, working hours, etc. So...
Is the PR text & any clarifying comments ever used to generate technical documentation?
Dedicated rotating engineers w/ or w/o knowledge can review and close.
We also have dedicated daily “office hour” for code review to have discussions