An author’s contract makes the pull-request author responsible for preparing a reviewable change and responding to feedback without treating it as a personal attack. section 2.4.2
Fulfill the contract by:
- Acting as your own first reviewer: anticipate reviewers’ questions and answer them proactively. section 2.4.2
- Making the pull request manageable: keep it short, atomic, and filled with context. section 2.4.2
- Making the pull request understandable and checking that every required item is complete before review. section 2.4.2
- Focusing on feedback and code improvement rather than on yourself or the reviewer. section 2.4.2
Figure 2.10 Guidelines for creating a feasibly-portioned PR (AKA how to make your PR manageable)
Use the PR-portioning guidance to judge whether a change is small and manageable. section 2.4.2
Figure 2.11 A context-filled, properly completed, and ready-to-view PR
Use the ready-to-view PR example to check context, completion, and review readiness. section 2.4.2
