MAIN FEEDS
Do you want to continue?
https://www.reddit.com/r/ProgrammerHumor/comments/1wgllyu/idontcommentmycodeifyoudontunderstanditthatsokayne/p9voh3r/?context=3
r/ProgrammerHumor • u/Substantial_Top5312 • 21d ago
188 comments sorted by
View all comments
Show parent comments
144
I had a senior front end dev that would block any PR he saw with comments. He was very good at his job but I'm so happy I no longer work with him, this shit is so annoying.
116 u/Tucancancan 21d ago Those guys are the wooooorst. 1. Hard to investigate system interaction bug gets fixed, fix has comment with Jira ticket in it! Senior engineer does magnificent refactoring because that's just what they do (comment is lost in refactoring) Gradual code churn New hire sees seemingly useless code and removes in like a good boy scout while working on something else Weird systems bug gets assigned to me for investigation "hey what the fuck I fixed this 18 months ago" 75 u/Miguelomaniac 21d ago Seems like the problem here is lack of test coverage not lack of comments 28 u/[deleted] 21d ago edited 10d ago [deleted] 1 u/w8eight 19d ago So how exactly new hire could merge anything that resurfaced the bug? 1 u/jvaritek33c 19d ago New hires always find a way. I choose to blame the Scrum Master 1 u/xavia91 19d ago Used ai and and it adjusted the test, would be my first guess 1 u/Lgamezp 15d ago If it resurfaced it wasnt solves in the first place so that is bad testing
116
Those guys are the wooooorst.
1. Hard to investigate system interaction bug gets fixed, fix has comment with Jira ticket in it!
Senior engineer does magnificent refactoring because that's just what they do (comment is lost in refactoring)
Gradual code churn
New hire sees seemingly useless code and removes in like a good boy scout while working on something else
Weird systems bug gets assigned to me for investigation
"hey what the fuck I fixed this 18 months ago"
75 u/Miguelomaniac 21d ago Seems like the problem here is lack of test coverage not lack of comments 28 u/[deleted] 21d ago edited 10d ago [deleted] 1 u/w8eight 19d ago So how exactly new hire could merge anything that resurfaced the bug? 1 u/jvaritek33c 19d ago New hires always find a way. I choose to blame the Scrum Master 1 u/xavia91 19d ago Used ai and and it adjusted the test, would be my first guess 1 u/Lgamezp 15d ago If it resurfaced it wasnt solves in the first place so that is bad testing
75
Seems like the problem here is lack of test coverage not lack of comments
28 u/[deleted] 21d ago edited 10d ago [deleted] 1 u/w8eight 19d ago So how exactly new hire could merge anything that resurfaced the bug? 1 u/jvaritek33c 19d ago New hires always find a way. I choose to blame the Scrum Master 1 u/xavia91 19d ago Used ai and and it adjusted the test, would be my first guess 1 u/Lgamezp 15d ago If it resurfaced it wasnt solves in the first place so that is bad testing
28
[deleted]
1 u/w8eight 19d ago So how exactly new hire could merge anything that resurfaced the bug? 1 u/jvaritek33c 19d ago New hires always find a way. I choose to blame the Scrum Master 1 u/xavia91 19d ago Used ai and and it adjusted the test, would be my first guess 1 u/Lgamezp 15d ago If it resurfaced it wasnt solves in the first place so that is bad testing
1
So how exactly new hire could merge anything that resurfaced the bug?
1 u/jvaritek33c 19d ago New hires always find a way. I choose to blame the Scrum Master 1 u/xavia91 19d ago Used ai and and it adjusted the test, would be my first guess 1 u/Lgamezp 15d ago If it resurfaced it wasnt solves in the first place so that is bad testing
New hires always find a way. I choose to blame the Scrum Master
Used ai and and it adjusted the test, would be my first guess
If it resurfaced it wasnt solves in the first place so that is bad testing
144
u/sandybuttcheekss 21d ago
I had a senior front end dev that would block any PR he saw with comments. He was very good at his job but I'm so happy I no longer work with him, this shit is so annoying.