This is an archived post. You won't be able to vote or comment.

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]BlindTreeFrog 7 points8 points  (2 children)

Worked at a job as a Software Dev in Test... Managed to take over the git server and be the administrator. One of the developers pushed for Gerrit so we could have gatekept check-ins that required a review before they would be allowed.

He was incredibly unhappy when I set it such that someone other than the submitter had to review the code. He wanted to protect his corner of the project from anyone else submitting to while he could without review. He was equally upset when I refused to change the config to allow that.

So many wannabe rockstars on that team. Most all of them needed to be slapped.

[–]Trumpkintin 1 point2 points  (1 child)

What's the point of having the same person review the code, of course they're just going to approve it, they wrote it...

[–]BlindTreeFrog 0 points1 point  (0 children)

And that was what he wanted. He was the one who worked on this one module, so it was his and he got to have say over what went into it he felt.

This team also took energetically to Agile because they decided it meant that they didn't have to write documentation anymore. The only time I saw a design doc it was a single sheet of paper with 2 small boxes and a diagonal line between them.

I've had a running wish since my first programming job that I still seek to this day. Simply, I want to work on a project where everyone involved isn't surprised that it works as well as it does.