Comment by tcmart14
12 hours 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.
Now 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.
As quoted from my comment: "My manager is also 100% with this."
And he has bosses. And are they okay with some whatever being an obstacle to work getting done? Probably not.
4 replies →
... Poor branch names?
Yea, 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.
[flagged]
Coupled with the other comment of yours that was flagged, the tone of this is completely unacceptable.
Maybe consider some anger management therapy? Nothing you’re talking about is in any way acceptable. Physical violence at work? You can’t be serious.
Its 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.
[flagged]
2 replies →