19
jhole89
7y

Looking through our gitlog today and see 3 PR's from our "lead developer". 2 of these were removing a single blank line from a class, and the 3rd was adding one back in. None of these had any title or commit messages on the PR's. This is a guy that talks down to everyone and deliberately makes other devs feel insignificant, saying he's too busy to write documentation and it's not needed because his uncommented code is self documenting. But hang on he's not too busy to waste time with pointless non-functional PR's that only remove a couple of blank lines? Scratching my head in disbelief that some devs think they can get away with shit like this. How about you drop the ego and actually try and work in collaboration with the other devs.

Comments
  • 1
    How do these folks get into senior positions then?
  • 2
    @freakko not senior, "lead" (flat hierarchy). He sold the management on his approach being the correct one and they've let him run with it. The stupid thing is we have more experienced devs than him that are being told to take a back seat even when we raise design concerns.
Add Comment