Re:Absolute truths I unlearned as junior developer
by gmueckl on Friday June 07, 2019 @01:14PM
(#20126720)
This is why I will point out issues in code only on the form of stating potential improvements as best as I can. I especially try to avoid hating on the author - it might have been me or the boss who's standing nearby...
In fact, I often conciously refrain from using blame on an "interesting" piece of code because it doesn't matter who wrote it. Looking it up would just satisfy idle curiosity, but yield noninsight into how to improve the code. In fact, I think that blame should just list the commits, but hide the authors by default. "What changed? " and "how?" are always much more pressing questions than "who?".
Re:Absolute truths I unlearned as junior developer
by Macha on Friday June 07, 2019 @01:57PM
(#20127113)
My experience varies a lot. Maybe it's just my colleagues write better commit messages but blame (and looking up the PR and code review) is often a good method of understanding why the code is that way
Re:Absolute truths I unlearned as junior developer
by gmueckl on Friday June 07, 2019 @06:24PM
(#20129185)
Yes, but blame puts the focus too much on the authors instead of the changes themselves. I also use it to understand how code evolved over time, but only in circumstances where I suspect the history to hold import clues.
Pinning bad work on a person does not make progress. Fixing bad code does.
Re:Absolute truths I unlearned as junior developer
by retiredcoder on Friday June 07, 2019 @02:53PM
(#20127565)
I am coming back from vacations next week and have 2 PRs to finish (couldn’t merge before because they are high risk), I am antecipating a lot of pain just to pick them up where I left off... I would rather do my tax returns instead lol
Re:Absolute truths I unlearned as junior developer
by tigershark on Friday June 07, 2019 @06:19PM
(#20129144)
It always happens to me while I approve my own pull requests to master.. :’(
This happened to me just yesterday! I was helping a co-worker with a problem, and I noticed some redundant code in the same function, so I told him he could simplify it while he was there. His response was, "...but you wrote that". (And as it turns out, only a few days prior!)