+1 (non-binding) Thanks Parth
On 7/21/15, 10:24 AM, "Gwen Shapira" <gshap...@cloudera.com> wrote: >+1 (binding) on using PRs. > >It sounds like we need additional discussion on how the transition >will happen. Maybe move that to a separate thread, to keep the vote >easy to follow. > >On Tue, Jul 21, 2015 at 4:28 AM, Ismael Juma <ism...@juma.me.uk> wrote: >> Hi all, >> >> I would like to start a vote on switching to GitHub pull requests for >>new >> contributions. To be precise, the vote is on whether we should: >> >> * Update the documentation to tell users to use pull requests instead of >> patches and Review Board (i.e. merge KAFKA-2321 and KAFKA-2349) >> * Use pull requests for new contributions >> >> In a previous discussion[1], everyone that participated was in favour. >>It's >> also worth reading the "Contributing Code Changes" wiki page[2] (if you >> haven't already) to understand the flow. >> >> A number of pull requests have been merged in the last few weeks to test >> this flow and I believe it's working well enough. As usual, there is >>always >> room for improvement and I expect is to tweak things as time goes on. >> >> The main downside of using GitHub pull requests is that we don't have >>write >> access to https://github.com/apache/kafka. That means that we rely on >> commit hooks to close integrated pull requests (the merge script takes >>care >> of formatting the message so that this happens) and the PR creator or >> Apache Infra to close pull requests that are not integrated. >> >> Regarding existing contributions, I think it's up to the contributor to >> decide whether they want to resubmit it as a pull request or not. I >>expect >> that there will be a transition period where the old and new way will >> co-exist. But that can be discussed separately. >> >> The vote will run for 72 hours. >> >> +1 (non-binding) from me. >> >> Best, >> Ismael >> >> [1] http://search-hadoop.com/m/uyzND1N6CDH1DUc82 >> [2] >> >>https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Code+Chang >>es >