Re:VS Code inserting 'Co-Authored-by Copilot' into commits r...
by dmitriv on Sunday May 03, 2026 @02:44AM
(#47994074)
If what you described would make it to our PR queue, it would definitely not pass the gates.
The idea was to track AI-only changes and add the trailer when such changes were detected AND the setting was enabled. Obviously, we didn't want to attribute all changes to AI. There is a bug in change detection (which slipped through testing), which led to even non-AI changes being tracked. And thus we have this problem.
The PR linked here wasn't even implementing the feature, it was changing the default for the setting.
Re:VS Code inserting 'Co-Authored-by Copilot' into commits r...
by detaro on Sunday May 03, 2026 @05:39AM
(#47995211)
> (which slipped through testing)
In another comment you say you caught it in testing and didn't think it needed fixing, which is it?
Re:VS Code inserting 'Co-Authored-by Copilot' into commits r...
by fastasucan on Monday May 04, 2026 @01:45AM
(#48005060)
>If what you described would make it to our PR queue, it would definitely not pass the gates
It just did though. Did you approve the PR without actually looking at the code?
To me, “let’s add false attribution to every commit by default without informing the user” falls squarely into that category. I don’t think I’ve ever worked in an environment where something like that wouldn’t have been red-flagged in three seconds by anyone who took even a casual glance. I’d honestly be embarrassed if such a proposal even made it into a public pull request for my organization, nevermind that pull request getting merged.