undefined | Better HN

0 pointslrem10y ago0 comments

It is mandatory that each code change is inspected for correctness, language style and approved by a code owner (all 3 may be the same person, or require multiple, depending on the situation).

0 comments

amelius10y ago

But I guess the original programmer can never be the same person as those other 3 roles? :)

mvgoogler10y ago

You always need to get at least _one_ other engineer to review your code.

OWNER and readability approvals may require additional reviewers but not always.

lremOP10y ago

Depends if his co-workers lock their screens ;)

j / k navigate · click thread line to collapse