Hi Rohit, Thanks for sharing your observations. I agree with you that anybody that contributes (or uses) Apache CloudStack should feel welcome in the community.
In the past weeks I tried to respond to as many PRs as I could and merge them as soon as it met the requirements. When I can not review myself, I usually ping someone and ask to do the review. This way I hope new contributors too get a fast response. I agree this is important so let us all have a look at the PRs on a regular basis and review them if possible. Also, I dropped the closing of PRs for now. I get your point and propose to leave it as-is. It’s not important now. I see a master that gets more stable every day and I think that’s awesome. Also for new contributors or new users this is great, as most new devs will checkout master to see if it is any good. In the current state, I’d say we improved greatly. So, it’s true that our review process takes time and effort, but IMHO it’s worth it. Running tests before anything hits master shows us problems in advance. And in case it doesn’t, we can always go ahead and improve the tests. If you see things we can improve, please share. Regards, Remi > On 26 Aug 2015, at 12:20, Rohit Yadav <bhais...@apache.org> wrote: > > Hi all, > > I’ve identified few issues around the recent changes that I don’t know > how we can fix or improve but I hope to get feedback from the > community. > > I understand that you may disagree with what I’m sharing which is > alright, even in your disagreement I hope that you don’t take an > offence on that, that is certainly not my aim. I personally want to > the community to be felt welcoming so that we can attract and retain > new contributors and have a nice environment for everyone. > > Some observations and comments: > > - Generally those of us who have worked for long in the community or > have colleagues from dayjob working in the ACS community - we have > better chances in getting their PRs merged; for new contributors this > pattern is not encouraging and certainly not welcoming if their PRs > get closed. > > - The recent drive to use Github PRs seems to be really working great > for us, but still the requirements of at least 2 +1s/LGTM, unit tests > and pedantic bike-shedding has cost us new developers/contributors and > kills the joy of programming for some; for experienced and > commercially backed developers this makes sense. I personally try to > be pragmatic and lenient on code reviews as long as smoke tests > (Travis) pass. > > - Contributions are process-oriented that cost time and effort; there > have been initiative to satisfy the tool but not the human, and not > optimizing on developer time. > > - What should we do to get more developer contributors and how to > attract hobbyists or casual contributors. > > Regards.
