Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing:add text around drive by contributions and maintainer time #112

Open
lwasser opened this issue Feb 4, 2025 · 0 comments
Open

Missing:add text around drive by contributions and maintainer time #112

lwasser opened this issue Feb 4, 2025 · 0 comments

Comments

@lwasser
Copy link
Member

lwasser commented Feb 4, 2025

          I went through a few more projects than I thought before I found a real life example. I knew it wasn't common, but thought I had seen explicit reference to typos before somewhere...

https://github.com/rails/rails/blob/main/CONTRIBUTING.md#did-you-fix-whitespace-format-code-or-make-a-purely-cosmetic-patch

There is a linked comment that expresses the reasoning well. I think the rational from elsewhere that I've seen is that busy maintainers want to cut down on drive-by contributors, who post the PR not because they use the project but because they are playing the github game. Or noisy contributions, where the author will submit a separate PR for each typo correction. I especially recall seeing this bemoaned every time "hacktoberfest" comes around.

Originally posted by @ucodery in #105 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant