Hacker News new | ask | show | jobs
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.