Hacker News new | ask | show | jobs
by quesera 1470 days ago
I disagree.

Spelling mistakes should be fixed.

A PR is click-click-done, whereas a textual description of the mistake requires small but real effort on the part of a project member.

And absolutely no one gives a flying crap about visibility in the changelog.

1 comments

I agree that spelling mistakes in a readme / website / docs should be fixed. Comments, on the other hand, are irrelevant, and not worth pinging the maintainers for unless they are affecting comprehension (not the case here). On a project with 0 PRs open it may be ok, but in more active projects that have dozens/hundreds of PRs at any given time, it's just adding noise.

It's also been a trend in the past few years for people to make this kind of PR, which is very low hanging fruit, just to increase the number of projects they "contributed to" and their profile activity. The author may have not had this intention, but it sure plays into the stereotype.