Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
-
Protect your master branch so future direct merges can not be done without a pull request. Also lock down who can do those merges and make sure his not on the list, if possible ( depends on your teams flow but usually is a good idea ) you could even enforce code reviews on pull requests to atleast get someone else on the team go over it before it can be merged.
Will slow him down and always have an extra set of eyes on the code being merged. -
yea, try to solve a social problem by technical solutions.. won't work. You wouldn't have stopped me, in this situation. And if you actually stop him, he will leave the team. Do you want that? Do the things he does, work?
You said you talked to him about it and it didn't work. Try it again and again, and tell him exactly what you think isn't a good idea and tell him exactly why. That's what made me a better teamplayer. -
Very difficult situation. What you're describing is pure technical debt. This code works... for now. I've been on a project where we had an exact person like that, and 3 years later we were still finding things that we didn't understand properly and we had to refactor because they no longer work correctly and no-one understood them.
The struggle is that managers and decision makers will see his output and go "It's fine!" and it'll only be later on that everyone will be banging their head and going "Why did we let this happen?"
A limited scope technical approach won't work (as someone has already said). -
@TerriToniAX @CrankyOldDev could you please not refer to me as "someone", thanks.
-
@plusgut My apologies. I was just too lazy. Won't happen again.
-
@plusgut Really? I couldn't remember the username of the comment by the time I had typed my reply. And I dislike jumping back and forward. I've raised the issue of context when replying in devRant a few times (once formally on the issues list). No offence intended.
-
@zshh You have an unmaintainable product. If he gets hit by a bus (or more likely, a bus arrives and takes him to a new job) then you're screwed. None of you will work there forever, so think about what happens if he leaves and the rest of you are left to work on his code.
Related Rants
Well... Erm...
My colleague is what you would call a cowboy coder. He solves problems with really complex solutions that only he understands and does not seem to care about that the team doesn't understand it. He's super fast and very skilled, but it leaves the rest of the team hanging. He sometimes works at his spare time so things we worked on the previous day can be totally changed the next day without any notice. He has also removed code written by someone else because he did not like it, in secret. I found this while browsing through commits that were committed directly to master without a PR.
We have tried talking to me about this but it doesn't seem to work. He seems to value speed over anything else and doesn't seem to have any respect for other team member's opinions.
What the hell do I do? Has anyone else worked with a similar typed person? He's really making my life hard and I think it's very frustrating. Please help.
undefined
cowboy coder
help