5 ms·
I respectfully disagree with you. I wasted perhap 10 hours of my life or more on this PR (pointlessly rebasing it a few times). 95 other PRs are outstanding. Th
by RayDonnelly 8y ago
I respectfully disagree with you. I wasted perhap 10 hours of my life or more on this PR (pointlessly rebasing it a few times). 95 other PRs are outstanding. There are many ways to improve this situation. I pointed two of them out:
1. Declare that you won't accept (or even review) them, preferably at the top of README.md *
2. Give a second person merge status.
I have some empathy for the maintainer but I also have empathy for the prospective contributors who spent a chunk of their free time working on https://github.com/ggreer/the_silver_searcher/pulls https://github.com/ggreer/the_silver_searcher/pulls
I've done my bit here to stop wasting people's time globally, I don't consider a maintainer's time more valuable than a contributor's time. That's elitest nonsense.
https://github.com/ggreer/the_silver_searcher/issues/1284 https://github.com/ggreer/the_silver_searcher/issues/1284
* The likely side-effect is a unblessed fork (or people not wasting their time, or if they do, well, they didn't read the README.md and cannot be too annoyed at anyone but themselves).
- ggreer 8y agoCome on dude, I just want to enjoy my long weekend. Please stop bothering me by creating spurious issues on my GitHub repo.
- RayDonnelly 8y agoOK sure, but would you consider merging one PR that explains that people should not expect PRs to get reviewed? I'm happy to do the work if you'll review and merge it. That way we can avoid unfortunate incidents such as this in future.