The Play of Pull Requests: Crafting Reviewable Code Changes

Saša Jurić's talk at Goatmire Elixir Conf transformed code review into a compelling narrative. He highlighted the common problem of unwieldy pull requests (PRs), leading to superficial reviews, security risks, and unmaintainable codebases. The key takeaway: reviewable PRs should ideally take 5-10 minutes to review, ideally under 300 lines of code. This is achieved by crafting concise, story-telling commit messages that clearly explain the rationale and steps of each change. Breaking down large features into smaller PRs and utilizing tools like `git fixup` to maintain a clean commit history are crucial for efficient code review and higher quality code. The talk emphasized that saying "I don't understand" is better than a meaningless "LGTM."