These are chat archives for fossasia/open-event-server

2nd
Jun 2018
Bhavesh Anand
@bhaveshAn
Jun 02 2018 11:06
@maxlorenz @schedutron @Kreijstal @niranjan94 @poush PR fossasia/open-event-server#4825 is ready for review. Please review.
Mario Behling
@mariobehling
Jun 02 2018 23:13
@bhaveshAn @maxlorenz @schedutron @Kreijstal @niranjan94 @poush @SaptakS We have a lot of open PRs in the server project. How could we speed up the merge process and solve smaller issues more quickly step by step https://github.com/fossasia/open-event-server/pulls ?
Saurav Tiwary
@srv-twry
Jun 02 2018 23:23
@mariobehling PRs such as fossasia/open-event-server#4829, fossasia/open-event-server#4810 and fossasia/open-event-server#4823
have already been approved by 2 or more than 2 reviewers.
I think the way to deal with long PR queue would be to give the merging rights to the students. Of course that would require more responsibility from the students but I think that's the way to go.
We won't be merging our own PRs. We would be merging PR which have been approved by atleast two fellow developers or mentors. We can leave the critical PRs for the approval of the mentors but the smaller and the trivial ones can be merged by us.