Ranter
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
Comments
-
ymas4717yYou shouldn't govern by fiat. It is much easier to change the habits of juniors if you let them figure it out. You should never assume what you say will be interpreted the way you mean it. The statement you made is quite generic and open to interpretation, if one of the juniors was insecure, he would have taken it as a criticism, hence the extreme end result. I believe you were advising your team to minimise branching and if the issue required an upstream change, they should seek further assistance. Remember, they're juniors, they're too scared to mess around with code that "works".
-
@ymas hmm, I'll take care of this in future. Actually he developed this service under direction of another senior dev. That guy left company because he was frustrated with management. So, in those times, he just approved every code-review without much attention.
There code is very buggy, and everytime a new bug gets filed, he put another condition-check in codebase. Now I'm assigned senior dev for that service, so as a starter I gave so called 'guidelines'.
I'll keep your advice in mind... This is going to be tough journey.😓 -
@thatJavaGuy are you looking for any (remote) freelancer, I'm not senior nor junior 😂
Told juniors about coding guidelines that don't put another if-else just to fix a bug. Think through about it and see if you can come up with better solutions.
Today one bug was filed, they asked what happened, one junior said that he [my name] asked me for no if-else in code. He kinda deleted all if-else in codebase and started using same implementation for everything.
I'm standing with a WTF face.
😐
devrant
shitt
this is not what i meant
coding guidelines
i don't know what to say😳