Don't attribute to malice what can be explained by incompetence. This is a bit on the harsh side in our field, nobody knows everything, and the knowledge avail

Don't attribute to malice what can be explained by incompetence. This is a bit on the harsh side in our field, nobody knows everything, and the knowledge available in a team is unevenly distributed, so, be kind and remember that while they might not know what you know.. You probably also don't know what they know.. For a new hire, however, they might be green, or have a different perspective that may need adjustment. They may have come from some consulting gig where they were taught to "just hack it, and fast!" rather than making quality software for the future, if so.. Don't be mean to them, but make sure to point out everything possible in their review so they have a chance to see the standards to which they must rise. When I'm faced with a review where I feel like everything is wrong, I will still comment on everything, but I will not be angry or agressive, but simply state what needs chage and why. This takes a long time, but I will log that time on their ticket.