Apologies for being nonspecific, but I don't know how else to describe Bob's struggles. Bob has been on the team for over a year now, and his code is just... not okay.
To his credit, he can make something that works... but that's not enough. His code belongs on programming horror. He's not supposed to be my junior; I'm just the repository's lead. I spend half my week helping him. Reviewing his pull requests takes hours because it's always a rats nest that needs significant refactoring/simplification. I'd love to say "do better" - but this is his best.
Most recently, Bob crashed his dev environment with a getter. (A mix of nested parsing logic with Angular's change detection = CPU crashed). It'd be impressive if it wasn't so irritating since I've already had a conversation with him about proper use of getters/setters. I even demonstrated how spammy the calls can be with a console.log statement for emphasis.
I could go on, but this is enough of a rant. I don't really know how to handle him going forward; I spend so much time helping and teaching him but he retains none of it.
Is there any hope for him? Any learning material? Advice on balancing my own sanity and workload?
The bar is even lower for a government environment. 🥲
Thankfully, we're already setup with a linter/auto code formatting. Mandatory PR templates/size limits would probably be overkill with a team this small - but I get your point. He really just needs to stick to the ticket's original scope and I'm going to harp on it more. We just had another talk about it on Thursday.
I appreciate the suggested dialogue. We've had conversations like this - mainly to see his openness to a lead in developer-adjacent things like testing. He's not very... self aware or well experienced career wise to say the least.
We're at the "figure out somewhere he can fit" or "remove him from the team" phase. We'd prefer to find a way to keep him.
Thank you for the sound advice/suggestions though.