Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

Where we’re at:
PR process comes with approvals, organizationally enforced (not structurally enforced but organizationally enforced), on main branches.

Where I think we need to be:
Whole team needs to be proactive at code review.

Code review post-implementation (code inspections).

Ideas:
Code Review board? List PRs with description? Real or virtual.

Board should follow google’s Google’s change lists? https://github.com/google/eng-practices/blob/master/review/developer/cl-descriptions.md

...

“Secondary reviewers” - please review this from lens X

Examples.:

Kick-off optional code inspections? Maybe a brown-bag-lunch code review process? Choose a method or basic process and just simply show the code on the main screen.

...