my first real open source PR got merged after three rounds of review that mostly humbled me

Thought the fix was clean going in. Maintainer asked why I'd chosen that approach over a simpler one already used elsewhere in the file, which I genuinely hadn't checked. Rewrote it twice, got merged, and now I actually read surrounding code before touching anything instead of just solving the immediate problem.