logoalt Hacker News

smooctoday at 5:26 AM5 repliesview on HN

So the fixes are still fixes, but we (I am also a OSS maintainer) are unwilling to accept them as they boost the contributor’s status where we think the merit is very or extremely limited.

Why not have these PRs counted differently (by the platform), and/or colored differently in the timeline(s) thus made less visible or more clear?


Replies

Arainachtoday at 5:36 AM

Change is bad unless it's great.

Unless the change is an obvious improvement, it has to be worth the time for the maintainers to spend attention reviewing it (and supporting the code forever, and all the rest).

Even if these particular changes are "harmless" and easy to review, accepting them sets a precedent that encourages an unsustainable flood of AI-generated changes that will overwhelm the project.

show 1 reply
21asdffdsa12today at 6:11 AM

These fixes, regularly destroy the architecture, accrue bloat for little gain, refuse to rewrite while demanding to rewrite- and many other such funny noises. Most code contribution by LLMs is garbage if you long-term care about the project. Look at closed source projects that ingest all this madness - windows with its seconds to open the explorer and other catastrophes.

bwhiting2356today at 5:38 AM

Why not let them have the status boost? This isn't zero sum.

show 4 replies
hypfertoday at 5:31 AM

The whole idea of "counting PRs" as a vanity metric is flawed because vanity metrics are flawed.

I don't think that there is a technical solution to be found here, as the problem is anything but technical.

__

A hack/trap:

Comment "Ah yes thanks a lot for the hint :)", then make the changes yourself.

Then see how the person reacts to that.

Hack the grifters. Hack the planet.

show 3 replies
Uptrendatoday at 6:08 AM

I think what you're saying is fair, tbh. Imagine you put in months or years into a project to make it a quality piece of work. It develops into something notable and you took all the risk. Then someone comes along to fix a spelling error with a pull request so their name effectively appears on the repo as a "contributor." And you just know right after its going on their resume as "contributed to [...]" or maybe if they're bold "software engineer working on [...]" which implies substantial investment. Then you're effectively sharing credit for YOUR work with someone who did nothing. That is rage inducing. ((Of course: it probably is just juniors trying their best in this horrible industry.))

On the other hand: lets be careful not to dismiss valid but inexperienced attempts to contribute. Having someone want to genuinely contribute to your software is incredibly generous. If someone seems like they're trying its better to give advice than act like a snob because its not good enough. Often pull requests only need small fixes to get in, anyway.