4 ms·
Just hit "deny." I've been training my co-workers that the AI-isms their "AI Assisted coding" do that some shit just isn't acceptable. I leave a comment and hit
by tcmart14 2mo ago
Just hit "deny." I've been training my co-workers that the AI-isms their "AI Assisted coding" do that some shit just isn't acceptable. I leave a comment and hit deny. It also helps that I control the policies on the repo and they can't merge in with a denial from anyone. So it's either a fix it, or explain why your work isn't getting done. My manager is also 100% with this.
I've denied for poor branch names and commits from AI. I've denied for too verbose of comments from AI. I've denied for parts of the code base being touched that are not relevant to the case they are working on (login isn't broken, your case is to add a check box in the settings pane, remove the changes made to login).
Pre-AI I wasn't fine with PRs with multiple features and touching irrelevant areas of the code base. Why would I be fine with it because my co-workers got new toys? You want AI to refactor a part of the codebase? Fine. Separate branch, new PR, and in the description, present an argument for it. Don't shoe horn it into something else. Also, I'm not obligated to hit approved on shit.
I may also be a bit privileged because I can be a pain in the ass to whole team. I may be slower, but I've got the numbers, my code is creating way less bugs then my "fast more efficient" co-workers.
- select1 2mo ago[flagged]
- tcmart14 2mo agoIts not petty is, Im not changing our standards because people got hot new toys. These are code policies we've had. If your AI usage can not conform to them, that's on you, not me.
- dofm 2mo agoCoupled with the other comment of yours that was flagged, the tone of this is completely unacceptable.
- Sharlin 2mo agoMaybe consider some anger management therapy? Nothing you’re talking about is in any way acceptable. Physical violence at work? You can’t be serious.
- LPisGood 2mo agoNow suppose four of your teammates, each have three PR’s in the backlog and you have to explain to your manager that you’re the bottleneck.
- tcmart14 2mo agoAs quoted from my comment: "My manager is also 100% with this."
- select1 2mo agoAnd he has bosses. And are they okay with some whatever being an obstacle to work getting done? Probably not.
- tcmart14 2mo agoThat's on him to handle and it hasn't been a problem.
- select1 2mo agoWhen it's a problem, you should both expect to be unemployed fairly rapidly. Progress isn't going to stop solely because you want to pick 'excessively long comments' as a hill to die on.
- johnnyanmac 2mo agoYou're really insistent on project your personal biases onto a person and company you do not even know the identity of. Turns out not every team has the idea of "progress" being yet another iteration of "more code submitted = better code". Some people actually desire or need to make quality products.
- Sharlin 2mo ago"Progress"
- matheusmoreira 2mo ago... Poor branch names?
- tcmart14 2mo agoYea, its not one I exactly agree with. But it is what the team collectively as a whole said is something is important. So to the other comments acting like I am just being ridiculous. Everything I deny PRs on is something time and time again the team collectively agrees on when asked how code should look and the process and all of that. The team can choose at any time to change that and for some reason, enough people on the team when asked in a meeting if branch names matter a lot and if they should follow a certain format, its a yes.