Copilot edited an ad into my PR
411–420 of 704 posts
Re: Copilot edited an ad into my PR
#4128 years later, this is where we are. I'm honestly just stunned, it takes some real talent to run a company that does it as consistently well as Microsoft.
Re: Copilot edited an ad into my PR
#413Re: Copilot edited an ad into my PR
#414Unless you're big enough like Meta, Microsoft, etc.
Re: Copilot edited an ad into my PR
#415Tim from the Copilot coding agent team here. We've now disabled these tips in pull requests created by or touched by Copilot, so you won't see this happen again for future PRs. We've been including product tips in PRs created by Copilot coding agent. The goal was to help developers learn new ways to use the agent in their workflow. But hearing the feedback here, and on reflection, this was the wrong judgement call. W…
[flagged]
Re: Copilot edited an ad into my PR
#416Re: Copilot edited an ad into my PR
#417Earlier quoted context omitted.
> […] and also the way Claude injects itself as a co-author. > Seeing them is an easy signal to recognize work that was submitted by someone so lazy they couldn’t even edit the commit message. You can see the vibe coded PRs right away. I was doing the opposite when using ChatGPT. Specifically manually setting the git commit author as ChatGPT complete with model used, and setting myself as committer. That way I (and e…
"Why would I commit something written by AI with myself as author?" Because you're the one who decided to take responsibility for it, and actually choose to PR it in its ultimate form. What utility do the reviews/maintainers get from you marking whats written by you vs. chatgpt? Other than your ability to scapegoat the LLM? The only thing that actually affects me (the hypothetical reviewer) and the project is the qua…
Model information for traceability and possibly future analysis/statistics, and author to know who is taking responsibility for the changes (and, thus, has deeply reviewed and understood them).
As long as those two information are present in the commit, I guess which commit field should hold which information is for the project to standardise. (but it should be normalised within a project, otherwise the "traceability/statistics" part cannot be applied reliably).
Re: Copilot edited an ad into my PR
#418Re: Copilot edited an ad into my PR
#419Earlier quoted context omitted.
I don't quite see the benefit of this, personally. Whoever is submitting the code is still responsible for it, why would the reviewer care if you wrote it with your fingers or if an LLM wrote (parts of) it? The quality+understanding bar shouldn't change just because "oh idk claude wrote this part". You don't get extra leeway just because you saved your own time writing the code - that fact doesn't benefit me/the proj…
> Whoever is submitting the code is still responsible for it, why would the reviewer care if you wrote it with your fingers or if an LLM wrote (parts of) it? Maybe one day we can say that, but currently, it matters a lot to a lot of people for many reasons.
That was my point here, it is a false signal in both directions.
Re: Copilot edited an ad into my PR
#420Tim from the Copilot coding agent team here. We've now disabled these tips in pull requests created by or touched by Copilot, so you won't see this happen again for future PRs. We've been including product tips in PRs created by Copilot coding agent. The goal was to help developers learn new ways to use the agent in their workflow. But hearing the feedback here, and on reflection, this was the wrong judgement call. W…
[flagged]
You may not feel you owe $BigCoEmployee better (though chances are, said person is just as much a community member here as you and the other users slamming them are), but you owe this community better if you're participating in it.