Copilot edited an ad into my PR
491–500 of 704 posts
Re: Copilot edited an ad into my PR
#492Earlier 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…
It’s not about who wrote it, but about who is submitting it. The LLM co-author indicates that the agent submitted it, which is a contraindication of there being a human taking responsibility for it. That being said, it also matters who wrote it, because it’s more likely for LLMs to write code that looks like quality code but is wrong, than the same is for humans.
Re: Copilot edited an ad into my PR
#493Earlier quoted context omitted.
It’s not really ads, it’s more like "Sent from my iPhone"-style sentences at the end of PR texts.
"Sent from my iPhone" actually is an ad when it’s the result of default settings. Furthermore, the ads in TFA are for Raycast, but apparently it’s not Raycast doing the injecting.
brawndo - its what your brain needs
Re: Copilot edited an ad into my PR
#494Earlier quoted context omitted.
It’s not really ads, it’s more like "Sent from my iPhone"-style sentences at the end of PR texts.
I agree. It's not an advertisement, it's simply a piece of information about your particular choice of technology. -------------- Sent from HackerNews Supreme™ - the best way to browse the Y Combinator Hacker News. Now on macOS, Windows, Linux, Android, iOS, and SONY BRAVIA Smart TV. Prices starting at €13.99 per month, billed yearly. https://hacker-news-supreme.io
Re: Copilot edited an ad into my PR
#495Earlier 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…
You're quite right that the quality of the code is all that matters in a PR. My point is more historical. AI is a very new tool, and as such the quality of the code it produces depends both on the quality of the tool, and how you've wielded it. I want to be able to track how well I've been using the tool, to see what techniques produce better results, to see if I'm getting better. There's a lot more to AI coding than…
Re: Copilot edited an ad into my PR
#496Earlier quoted context omitted.
I think it depends a lot if you reviewed it as carefully as you would your own code. Of course most people don’t do that
In my project's readme I put this text: "There is no commit by an agent user, for two reasons: * If an agent commits locally during development, the code is reviewed and often thoroughly modified and rearranged by a human. * I don't want to push unreviewed code to the repo, so I have set up a git hook refusing to push commits done by an LLM agent." It's not that I want to hide the use of llms, I just modified code a…
Seems... Not that useful?
Why would someone make commits in your local projects without you knowing about it? That git hook only works on your own machine, so you're trying to prevent yourself from pushing code you haven't reviewed, but the only way that can happen is if you use an agent locally that also make commits, and you aren't aware of it?
I'm not sure how you'd end up in that situation, unless you have LLMs running autonomously on your computer that you don't have actual runtime insights into? Which seems like it'd be a way bigger problem than "code I didn't reviewed was pushed".
Re: Copilot edited an ad into my PR
#497Once again, Microslop doing Microslop things
Re: Copilot edited an ad into my PR
#498Earlier quoted context omitted.
I agree. It's not an advertisement, it's simply a piece of information about your particular choice of technology. -------------- Sent from HackerNews Supreme™ - the best way to browse the Y Combinator Hacker News. Now on macOS, Windows, Linux, Android, iOS, and SONY BRAVIA Smart TV. Prices starting at €13.99 per month, billed yearly. https://hacker-news-supreme.io
I'm curious about how a hacker news client on a smart TV would work...
Re: Copilot edited an ad into my PR
#499Re: Copilot edited an ad into my PR
#500Tim 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…
Just to add to the feedback. No one, anywhere, ever wants this or anything like it. Do not inject anything that is outside of the context of the session, ever. This is how you get your software banned at large companies. Question for you, did anyone on the team really not push back? Does the team really think anyone wants ads in their copilot output? If the answer to both of these is no, you have a team full of yes m…