No need to use github for reviewboard. reviews.apache.org, already exists. Of course, no reason not to use github in addition -several projects use Github pull requests as part of their preferred process.
--David Wed, Jul 17, 2013 at 2:15 PM, Isuru Perera <[email protected]> wrote: > I think it's a good idea to mirror the code to GitHub and set up ReviewBoard > or Gerrit to review patches. > > > On Wed, Jul 17, 2013 at 7:31 PM, Noah Slater <[email protected]> wrote: >> >> It is a good idea to accept patches in any medium: >> >> * Email submissions >> * ReviewBoard or Gerrit >> * JIRA tickets with attachments >> * GitHub pull requests >> >> (Yes, you can accept GitHub pull requests. Infra will mirror us to >> Github.) >> >> We can have ReviewBoard or Gerrit set up if we want to review patches >> internally. >> >> It is in our best interests not to exclude any form of contribution. >> Though, of course, we should communicate our recommended / preferred method. >> >> >> On 17 July 2013 02:18, Pradeep Fernando <[email protected]> wrote: >>> >>> Hi, >>> >>> Looks like, we don't have JIRA yet... >>> >>> https://issues.apache.org/jira/browse/INFRA-6427 >>> >>> >>> On Wed, Jul 17, 2013 at 6:43 AM, Pradeep Fernando <[email protected]> >>> wrote: >>>> >>>> +1. However, i was under the impression that JIRA is not set up yet.. It >>>> was the case, when i submitted a patch last time. >>>> >>>> >>>> On Wed, Jul 17, 2013 at 6:27 AM, Samisa Abeysinghe >>>> <[email protected]> wrote: >>>>> >>>>> $subject? >>>>> >>>>> List is better used for discussions. We should use Jira for patch >>>>> management. >>>>> >>>>> Thanks, >>>>> Samisa... >>>> >>>> >>>> >>>> >>>> -- >>>> Pradeep Fernando. >>>> http://pradeepfernando.blogspot.com/ >>> >>> >>> >>> >>> -- >>> Pradeep Fernando. >>> http://pradeepfernando.blogspot.com/ >> >> >> >> >> -- >> NS > > > > > -- > Isuru Perera > Senior Software Engineer | WSO2, Inc. | http://wso2.com/ > Lean . Enterprise . Middleware > > Twitter: http://twitter.com/chrishantha | LinkedIn: > http://lk.linkedin.com/in/chrishantha/
