This pattern is relatively common in most teams,
but it often goes unrecognized ๐: an engineer understands a problem ๐ก,
breaks down the project into smaller tasks ๐ช,
and submits code that has little room for improvement ๐ฆ.
Most likely, not all of the commits that make up the project will be Bullseyes.ย
But the ones that are, generally have a small to modest impact
and were thoroughly reviewed and approved on the first try. ๐ฅ
Celebrate them! ๐ฅณ
HOW TO RECOGNIZE IT
In practice, Bullseye Commits can be identified by when they were submitted
in regard to the deadline โ ๏ธ, their impact, and how they were treated in the review process.
Generally, the code was started and completed in advance โ of the deadline, with negligible churn.
The commitโs Impact was small to modest in size and was then thoroughly reviewed. ๐ต๏ธโโ๏ธ
It was approved on the first try. ๐ฏ
In Bullseye Commits, code reviews are substantive.โ ๐ช
WHAT TO DO
Recognize a clean bullseye in a stand-up ๐งโโ๏ธ, or a simple note:๐ท๏ธ
โI saw that check-in, nice job!โ๐ฃ๏ธ
Whether itโs public or private, showing that you noticed and that you care will only reinforce this pattern. ๐ช
If thereโs an engineer who regularly makes Bullseye Commits,
it may be helpful for others to understand how they approach projects.
Ask the engineer to do a lunch ๐ and learn, or consider asking them
to provide feedback ๐ค on another engineerโs work in the review process.