public inbox for [email protected]  

help / color / mirror / Atom feed

From: Tom Lane <[email protected]>
To: Peter Eisentraut <[email protected]>
Cc: Josh Berkus <[email protected]>
Cc: PostgreSQL-development <[email protected]>
Subject: Re: Considering Gerrit for CFs
Date: Fri, 08 Feb 2013 13:41:22 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgsql-hackers>

Peter Eisentraut <[email protected]> writes:
> On 2/8/13 10:58 AM, Tom Lane wrote:
>> ...  I don't object to people using their
>> tools-of-choice to perform reviewing, but we need some way of making
>> sure that the reviews get archived.

> Gerrit sends me an email every times something happens, so I think this
> is not going to be a problem.

As long as it can be persuaded to mail the text of reviews (not just a
notification), that'd probably be all right.

> What it doesn't support AFAICT is sending emails *in*, but I don't see
> that as a requirement.

Meh.  Many of the complaints about the current CF application boil down
to the fact that it doesn't accept email input, so I'm not really
convinced that a different tool that also doesn't accept email input
is going to be a big step forward.

			regards, tom lane


-- 
Sent via pgsql-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

view thread (35+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Considering Gerrit for CFs
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox