785
Better merge it, too!
(i.imgflip.com)
Welcome to Programmer Humor!
This is a place where you can post jokes, memes, humor, etc. related to programming!
For sharing awful code theres also Programming Horror.
My preferred variation of this is to make it an open question that leaves them in the position of authority, and assumes that they made a deliberate decision.
For example, instead of "Why aren't you using StandardLib that does 90% of this?", I would try "Could this be achieved with StandardLib? Seems like it would cover 90% of this".
If you have seniority and they are a junior, some juniors do respond well to a senior having more knowledge about the codebase. With them, it can be beneficial to use a tone like "We have library X that seems like it could do a lot of the functionality here, unless you already took a look?" I know it's like 90% of the same but I know people who will just be shellshocked and just blindly say "yes" to any question you ask them, and I don't want a blind "yes" I wanna know the truth :) it also lets then explain why they didn't use it if they have a legit reason because hey, maybe I'm the one who needs to be caught up