Practice catching risky changes before they ship.
Read a pull request the way a senior reviewer does: open the diff, find the line that turns into an incident, and decide whether it can merge. No terminal, no sandbox — just review judgment.
Open pull requests
A cleanup of request-context setup quietly changes how the tenant is resolved. Decide whether it is safe to merge.
A retry helper is trimmed down to remove some math. Decide whether the new delay logic is production-safe.
A cache-key helper is shortened. Decide whether the simpler key still keeps data separated correctly.
How a review is scored
Each review is graded objectively out of 100: flag at least one genuinely risky line (40), pick the right risk category (20), choose the right review decision (20), and name a real reason in your comment (20). After you submit, the expected risky lines and a Buglyst reference review are revealed so you can compare your judgment.