Jacques Nadeau 2013-02-25, 03:54
Julian Hyde 2013-02-25, 19:42
I also agree opening pull requests are a lot easy, then manually creating
and uploading patch files.
However the only annoyance is that we do need to remember closing them once
it's merged ourselves.
Closing a pull request is less work then patch files though, so I would
prefer what Julian suggested
On Mon, Feb 25, 2013 at 11:42 AM, Julian Hyde <[EMAIL PROTECTED]> wrote:
> On Feb 24, 2013, at 7:54 PM, Jacques Nadeau <[EMAIL PROTECTED]> wrote:
> > We need to switch to using patches rather than pull requests. Since
> > is a read only mirror of Apache's Git repo and we can't close the
> > there (as far as I understand), we need to move away from using pull
> > requests. (The only time we "closed" requests is when we took the
> > as is rather than doing anything such as rebasing; in those cases GitHub
> > detected the ff merge.) You can see  for more information.
> > So, from now on, please provide patches attached to your JIRAs. We also
> > have reviewboard enabled now at . For larger patches, we should
> > probably use that.
> But is it OK for me to put the URL of a github patch into the jira case?
> (Each github pull request is available as a patch, see e.g.
> https://github.com/apache/incubator-drill/pull/9.patch.) I would think
> so. This is the easiest way for me, as a non-committing developer, to
> submit code, and I don't think it makes your life more difficult.
Jacques Nadeau 2013-02-25, 20:28
Julian Hyde 2013-02-25, 20:40
Jacques Nadeau 2013-02-25, 21:33
Timothy Chen 2013-02-25, 21:39
Henry Saputra 2013-02-26, 19:46
Timothy Chen 2013-02-26, 20:54
Henry Saputra 2013-02-26, 21:02
Ted Dunning 2013-02-26, 21:50