-
-
Notifications
You must be signed in to change notification settings - Fork 27.9k
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
Project status #1363
Comments
@troxler Just because there are lots of open PRs doesn't mean this project is dying. This repo is just a list of what I consider good. The beauty of open source is that anyone can create a list. It doesn't have to be listed here. There are currently 1500 repos tagged with the I have tried adding more collaborators multiple times before without success. You don't even need to be a collaborator to help out. Merging the PRs are easy. Thoroughly reviewing them is the time consuming part, and anyone can help out with that, but few do. I'm also working with community members on a awesome linter which will hopefully lessen my PR review work. |
No it does not indeed. I've been following this project for a while and I think during this time, the number of open pull requests even decreased. But many of them were closed because their authors apparently lost interest and gave up or just did not answer anymore. I strongly think that is also has to do with the amount of time it takes from the request creation to actually being merged (or closed).
I do realize that (kudos and thanks btw!). And I especially get the "demotivating" part as I've seen many examples of authors that clearly did not read the guidelines at all.
That may be true. But don't you need tools to see at a glance which pull requests could potentially be merged (i.e. someone reviewed it and thinks it is OK) and which need more work? Do you go through all 100+ requests regularly just to check that (well, just the once that were updated recently but you get the point)? Wouldn't it be helpful to have appropriate labels or a being able to filter by approvals? Contributors without privileges can't use either. It would also make sense for the reviewers to quickly find requests that actually need a review without skimming through a long list first. And personally, I also find it demotivating to do reviews without proper tools. |
Yup, but in most cases it takes a while because people keep opening PRs without having put much effort into their list or PR, so they require a lot of review time, which means I have less time for other PRs, and suddenly there are 100+ open PRs...
I go through my Awesome repo notifications once in awhile to see if any new PRs are ready for another review.
Sure, this all sounds nice, but you don't just add random people to a project and hope they do a good job. You add people that already regularly do a good job.
All PRs need review, including already reviewed ones. Even when I get external review help, they miss a lot of things, so the more reviews of the same PR, the better. |
Yes I can imagine. First thing I would do is to close the PRs of authors that did not respond to any comments in months. It seems useless to let people do yet another review on those anyway.
That would be ideal yes. What currently is happening is the following: Some PRs (mostly newer ones) get multiple repetitive reviews while other PRs (mostly older ones) don't get any. My own PR has been open for exactly one year today and did not get a review. I tried pinging you, I updated the list several times to comply with updated rules, I fixed the merge conflict caused by #1343, I ran the linter and did some more updates, I did reviews on other PRs and pinged you again. Well, a year has passed since I created the PR and it is still open. How would anyone be interested to go through all that hassle just to be ignored? I highly appreciate what you do (here and on the many other projects you have) and I definitely understand that it all takes a lot of time. All I'm saying is that it does not seem to scale. Having that many open PRs of which many are so old does not attract new contributions in my opinion. That is all I wanted to say :-) |
Closing this as I'm back to merging PRs again. |
I got pretty burned out by this project and haven't been active for many months. This project is called Awesome and lists are expected to be of (awesomely) high-quality. Unfortunately, many submissions were not and required a lot of effort on my part to just comment on things that were already listed in the pull request guidelines. This doesn't scale to one person when there are many new submissions a day.
I'm ready to put some work into this project again, but I need your help.
There's a new requirement for submissions now. When you submit a new list, you're required to review at least 2 other open pull requests. This is my experiment to try to make the Awesome project self-sustaining. Read more here.
This requirement also applies to existing submissions, so if you like to see your submission merged, review at least 2 other pull requests and I'll prioritize getting your submission merged.
Please also go through the list guidelines again on your own submissions. You have probably missed something.
The text was updated successfully, but these errors were encountered: