4 ms·
Everyone working on a project is expected to chip in and review code, regardless of how junior or senior they are. Typically, a very junior engineer reviewing
by ozzythecat 5y ago
Everyone working on a project is expected to chip in and review code, regardless of how junior or senior they are.
Typically, a very junior engineer reviewing code for another junior engineer isn’t sufficient. We may require two sign offs before the code is merged.
Senior+ engineers review critical path code, but not every single code review. This is to create space for others and also ensure others are not merely spectators but actively own what’s going on.