Re: [CommitFest App] Feature request -- review e-mail additions

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: [CommitFest App] Feature request -- review e-mail additions
Дата
Msg-id 20160330174744.GA978880@alvherre.pgsql
обсуждение исходный текст
Ответ на [CommitFest App] Feature request -- review e-mail additions  (José Luis Tallón <jltallon@adv-solutions.net>)
Ответы Re: [CommitFest App] Feature request -- review e-mail additions  (Robert Haas <robertmhaas@gmail.com>)
Re: [CommitFest App] Feature request -- review e-mail additions  (Magnus Hagander <magnus@hagander.net>)
Список pgsql-hackers
José Luis Tallón wrote:

>     Just wanted to suggest two minor mods to the review e-mails
> auto-generated by the app:
> 
> * Prepend a [review] tag to the e-mail subject
>     ... so that e-mails sent to -hackers will read  " [HACKERS] [review]
> <patch title goes here>"

Changing the subject of an email causes Gmail to break the threads, so
anything in that line should be discouraged.  -1 from me.  I would be
happier if the subject of the submission email is kept intact, i.e. not
use the patch title that was used in commitfest app but use the one in
the email.  These often differ.

> * Auto-CC the patch author on this e-mail
>     I guess this should speed up reactions / make communication a bit more
> fluid.

Yes, strong +1 on this.

3) Have the auto-generated emails insert In-Reply-To headers (and
perhaps References).

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



В списке pgsql-hackers по дате отправления:

Предыдущее
От: José Luis Tallón
Дата:
Сообщение: [CommitFest App] Feature request -- review e-mail additions
Следующее
От: Tom Lane
Дата:
Сообщение: Re: [COMMITTERS] pgsql: Introduce SP-GiST operator class over box.