|
|
|
|
|
by tumetab1
2418 days ago
|
|
I think that matches perfectly what I have imagined. async and sync work support for PR reviews. a) People not currently working on a repo (don't have a local build environment setup) and do tweaks and suggest improvements (e.g. add a unit test to prove make a bug clear) in a async way. b) A tech lead/senior developer can request the author to join a live session to go explain the whole PR has a bad approach or misses a base guideline on the repo. Not sure on this case how a summary of review session should be added (maybe the PR should have a log of review sessions done). c) A teach lead takes the PR has an opportunity to ask the whole team to join a review so that he can point out a recurring issues on the repo or to just align ideas on code quality. |
|