|
Ruggan posted:[*] Rules lawyered me with a terrible logic argument about why he thinks he's "technically correct" about something, ending his argument with "Explaining this is tiresome and quite useless. I rest my case." What's his forums username
|
# ? Jan 6, 2023 06:22 |
|
|
# ? May 10, 2024 00:50 |
|
Ok so I wanna post more about this because it's literally the craziest exchange I've had with another dev... ever? Basically the background is that we have some class which we'll call BaseClass, and this class had one implementation which we'll call ImplClass. The guy's job was to fork this implementation and make a new implementation that did some things differently. Now, ImplClass had a comment above its declaration, which uselessly stated "This is the only production implementation of BaseClass". It's particularly useless because the tools at my job make it really easy to find implementations and references of classes in the codebase - seeing it's the only implementation of a base class is a click or two away. Anyway, when he forked ImplClass to make ImplClass2, he did a malformed find-and-replace and left the comment saying "This is the only production implementation of ImplClass2", so I left a little comment along the lines of "Hey, I think this comment is referencing the wrong class, but this comment is somewhat useless so maybe just remove it?". In response, the guy changes the comment to read This is the only production implementation of BaseClass - i.e. he reverts it to its original state. But hey, hold on, this isn't true anymore, because his new class is a second implementation of BaseClass that will eventually make its way to production (if all goes as planned). So I reopen the comment and say "this isn't accurate anymore because [reasons] but again, you can just remove the comment because I don't think it's contributing much". This time, the guy changes the comment to read This is NOT the only production implementation of BaseClass. So I reach out to him directly, to say that I think he's missing the intent of my review comment, and that he can just remove it and we're good, it's really not a big deal. But he can't just erase the drat comment and move on, he needs to justify his dumb change to add the word "NOT" in allcaps to the comment. So instead, we proceed to have this stupid as gently caress conversation over chat: quote:Him: That comment is in a lot of places. You said it wasn't helpful. Its opposite idea can't not be helpful by logic. "A is not helpful, not A, therefore is helpful". I didn't create the idea of adding that comment. That seemed too subjective to me. And one can't argue with logic. I wish I was making this up. Just delete the drat comment or literally write one sentence to me saying "I think this is helpful, actually" dude I don't care about your logic argument.
|
# ? Jan 6, 2023 06:53 |
|
prom candy posted:What's his forums username Alternatively, buy him an account.
|
# ? Jan 6, 2023 07:01 |
|
"I'll show YOU a valuable comment: 'You're acting like an rear end in a top hat!'!"
|
# ? Jan 6, 2023 07:23 |
|
Sorry, OP. Your boss hired Ben Shapiro.
|
# ? Jan 6, 2023 08:02 |
|
Sorry OP, you got DESTROYED by FACTS and LOGIC. How did he make it past interview?
|
# ? Jan 6, 2023 08:14 |
|
Ask him what his karma is on r/atheism.
|
# ? Jan 6, 2023 08:54 |
|
I don't suppose you tried to get him to remove the original useless comment too and thereby resolve the logical fallacy?
|
# ? Jan 6, 2023 09:00 |
|
Ruggan posted:You said it wasn't helpful. Its opposite idea can't not be helpful by logic. "A is not helpful, not A, therefore is helpful". this is terrible logic
|
# ? Jan 6, 2023 10:16 |
|
he'd be awesome at destroying rogue star trek computers though
|
# ? Jan 6, 2023 10:20 |
|
I mean I hope he isn't past his probationary period yet. Just get him kicked the gently caress out.
|
# ? Jan 6, 2023 12:20 |
|
Next PR removes all the existing comments.
|
# ? Jan 6, 2023 13:44 |
|
Tell him A is "him being bad at communication", therefore
|
# ? Jan 6, 2023 14:25 |
|
quote:Him: Arguing about this is very stressful and tiresome, and quite useless. Imagine trying to reason about subjective matter. I rest my case. Lmfao then stop arguing and just do it. My developer life has been so much simpler and less stressful since adopting the strategy of "accept every pr comment possible". You want the code upside down with a comment between every line? Sure, clearly I'm not going to win an argument against it here, have it your way and I'm moving on.
|
# ? Jan 6, 2023 14:58 |
|
Carbon dioxide posted:I mean I hope he isn't past his probationary period yet. Just get him kicked the gently caress out. Yeah honestly op just take this exchange to whoever is in charge of firing dumb motherfuckers and let them do their thing. edit: or quit
|
# ? Jan 6, 2023 16:05 |
|
It’s classic for people to not want to make the change you request because they are lazy, and then spend more time arguing about it then it would take to just do it.
|
# ? Jan 6, 2023 16:50 |
|
prom candy posted:Yeah honestly op just take this exchange to whoever is in charge of firing dumb motherfuckers and let them do their thing. This. This guy has missed the fundamental point that software development is a collaborative exercise. He's insisting on being right when he should be trying to understand the perspective of somebody who hasn't read the ticket he picked up and is only seeing the code (e.g. himself, 6 months from now).
|
# ? Jan 6, 2023 16:56 |
|
He sounds a bit like Elon Musk when he tried to write code at PayPal or whatever.
|
# ? Jan 6, 2023 17:26 |
|
Ruggan posted:I wish I was making this up. Just delete the drat comment or literally write one sentence to me saying "I think this is helpful, actually" dude I don't care about your logic argument. Wow, they're a real life 10x devloper
|
# ? Jan 6, 2023 17:58 |
|
This isn't much different than experiences I've had with code reviews in my organizations with a bunch of electrical engineers who did something they thought was novel and now think they're software kings. I've had a lot of butthurt in those code reviews. Somebody else here kind of gave up and just required code to pass multiple automated gates because these people were more amenable to a beep beep boop computer check than somebody saying that they're crazy if-if-else-if-else-if-else-else-if-else pile should be done recursively in a tenth of the size and actually work correctly.
|
# ? Jan 6, 2023 18:09 |
|
Either a concentrated example of big fish, little pond finally entering the ocean with the rest of us. ... That or he's on the spectrum.
|
# ? Jan 6, 2023 21:05 |
|
Dying on the worlds smallest hill, possibly sacrificing employment. This is “Confederacy of Dunces 2: Electric Code Review.”
|
# ? Jan 6, 2023 22:39 |
|
Carbon dioxide posted:I mean I hope he isn't past his probationary period yet. Just get him kicked the gently caress out. This really. You can teach someome to code and to code to your standards. You can't teach them not to be an rear end in a top hat.
|
# ? Jan 7, 2023 00:37 |
|
Carbon dioxide posted:I mean I hope he isn't past his probationary period yet. Just get him kicked the gently caress out. Indeed. Even if they’re a talented developer, if they can’t work in a team, there’s no point in keeping them. One of my highest criteria for keeping a new hire is seeing if they play well with others.
|
# ? Jan 7, 2023 02:10 |
Yeah dude needs a come to jesus talk with his manager about how to operate in a professional environment.
|
|
# ? Jan 7, 2023 17:48 |
|
More like a go to hell talk 🤣🤣🤣
|
# ? Jan 7, 2023 17:58 |
|
At this point I'd be like, "Okay, we've had the discussion, thank you for your input but my opinion is unchanged. I understand your point of view and I know that you disagree with me. However, I have seniority and I am formally overruling you. Please remove the comment, as I've requested." This changes the situation from a perhaps debatable, subjective, abstract inability to understand logical reasoning, or what good comments look like, to a refusal to follow instructions, which is a much more clear-cut indicator of something going wrong.
|
# ? Jan 7, 2023 20:39 |
|
Whatever happens OP please keep us posted, this is great content
|
# ? Jan 8, 2023 00:13 |
|
ChickenWing posted:Yeah dude needs a come to jesus talk with his manager about how to operate in a professional environment. I have some fear that either: 1. The manager is hoping somebody else will do it. 2. Doesn't believe in any of it themselves.
|
# ? Jan 8, 2023 03:02 |
|
Rocko Bonaparte posted:1. The manager is hoping somebody else will do it. A manager with no teeth is the worst manager
|
# ? Jan 8, 2023 03:51 |
|
Judge Schnoopy posted:Lmfao then stop arguing and just do it. Unironically this. Sure, it might make the codebase inconsistent and dependent on who reviews what PRs, but that’s a problem with the team as a whole - not your own. Takes the blame off of you.
|
# ? Jan 8, 2023 04:23 |
|
I've been in this story, it never gets better. My favorite part is when they say they don't realize how unpleasant they're being, but they're very sensitive about people giving any kind of correction to them. If you want/need to make this work: - Don't use text when there will be a back and forth, talk to them - Realize it will be more efficient for you to suggest the exact change you want to see - Avoid style discussions On the bright side, we've all learned that if it is rude to tell someone to go eat poo poo, it is polite to tell them to NOT eat poo poo. Logic!
|
# ? Jan 8, 2023 05:36 |
|
I am proud that my thread title has stood the test of time, butquote:Working in Development: Let me show you how logic is never wrong
|
# ? Jan 8, 2023 15:48 |
|
I'm in an alumni Slack org for an old job, and someone else who is now a software engineering director asked this absolute banger of a question:quote:How do your companies measure (if they do) engineering throughput or productivity? The best I can come up with is number of PRs (measured through a tool called Code Climate Velocity). Any other ideas? (This is the same director who, a few months ago, asked if anyone had experience with a baffling thing she was running into: She wanted her engineers to be on-call outside of normal business hours, and they wanted more money in exchange. Has anyone ever heard of anything like that??? It's so weird!!)
|
# ? Jan 9, 2023 15:31 |
|
Never inner source for another team. Ever.
|
# ? Jan 10, 2023 21:55 |
|
One weird trick that project owners love to make their schedule work out.
|
# ? Jan 11, 2023 03:31 |
|
Love Stole the Day posted:Never inner source for another team. Ever. What's this? An opportunity to have multiple bosses?
|
# ? Jan 11, 2023 03:36 |
|
For us, it’s when another team doesn’t have the capacity to pick up the work you need them to do, so you do it for them. Or when you fix bugs that impact you directly, rather than waiting for it to be prioritized by them.
|
# ? Jan 11, 2023 13:39 |
|
My current team expects everyone under the same umbrella organization to pull their weight on everything software, network engineering, graph modeling, device curation, and infrastructure/deployment as needed. We tend to be the catchall bucket for any work to unblock the org’s current pet project, so I know way more and call way more shots than I should. If poo poo ever breaks, you know who to blame 😋
|
# ? Jan 11, 2023 15:21 |
|
|
# ? May 10, 2024 00:50 |
|
smackfu posted:For us, it’s when another team doesn’t have the capacity to pick up the work you need them to do, so you do it for them. Or when you fix bugs that impact you directly, rather than waiting for it to be prioritized by them.
|
# ? Jan 11, 2023 16:14 |