Live data from Hacker News

Copilot edited an ad into my PR

notes.zachmanson.com

491–500 of 704 posts

Re: Copilot edited an ad into my PR

#492
post #485
post #393

Earlier 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.

Well if an agent is submitting it I'm just going to reject it, thats no problem. "Just send me the prompt".

Re: Copilot edited an ad into my PR

#493
post #475
post #361

Earlier 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.

companies pay for ad distribution. its not like they give a free ad service -$-. maybe they dont chose how the campaigns are done (and dont give shits)

brawndo - its what your brain needs

Re: Copilot edited an ad into my PR

#494
post #361

Earlier 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

I'm curious about how a hacker news client on a smart TV would work...

Re: Copilot edited an ad into my PR

#495
post #393

Earlier 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…

Yep other people pointed this out as well, this makes sense to me.

Re: Copilot edited an ad into my PR

#496

Earlier 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…

> * 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."

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

#498
post #494

Earlier 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...

You can try it now! Prices starting at €13.99 per month, billed yearly.

Re: Copilot edited an ad into my PR

#500

Tim 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…

They already know that nobody wants it. They don’t care.
Post reply on HN