News for hackers. Stuff that matters.
Re:Engineer says Google fired her for notifying co-workers o... by singron on Tuesday December 17, 2019 @12:35PM (#21816073)
If you need 2 approvals at Google, then usually 1 is for readability in a specific language. I.e. one of the reviewers is understood to mostly be looking at code style and not domain logic.

In my experience, you more often get the other phenomena where all reviewers (and some other people who get automatically CC'd) dogpile the review and you potentially have too many comments. Or someone ignores the review and it never gets approved. Cursory reviews were basically never a problem.

There are no comments on this comment yet.