Code Review

Code Review

Review expectations, approval process, and merge policy.

Review Process Overview

What Reviewers Look For

Correctness

Style and Conventions

Testing

Performance

Security

Approval Requirements

CI Requirements

Merge Policy

Squash Merge

Who Can Merge

Committer Responsibilities

Requesting Reviewers

Responding to Feedback

Backport Merges

See Also

• Contributing — How to submit a PR
• Continuous Integration — CI checks that must pass
• Coding Style — Code standards reviewers enforce