Do all the things like ++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatarSign Up
It looks like this dev doesn't know what is used and what isn't.
You should talk to him and explain that he's been doing useless work.
Tell him why, and point him to what needs to be done.
If you can't tell him, explain it to a direct manager, in a friendly way..
Don't make yourself the bad guy.
@forkbomber He's one of the senior developers and very aware that the service is no longer in use (another rant someday on his role of why the service is no longer used), which is why he chose to omit what code he 'refactored'. Not the first time he's glorified BS over real accomplishments and taken credit for others work.
I don't know making things better is never bad.
Maybe point out he's fixing things that where not broken?
ultrono24704ySounds like a complete of everyone's time. Either way I'd have to ask him face to face wtf he was thinking.
Surely there's more pressing things to be doing than reworking code years old that isn't used?
If he's a senior dev he definitely needs a slap. If he's a junior he needs some guidance imo.
Unless the said unnecessary refactoring affects something I was doing directly, in a negative way i personally wouldn't bring it up to a manager.