pull request email notification

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

pull request email notification

yzhdanov
Guys, why pull request notification goes to devlist, but not to
[hidden email]
?

--Yakov
Reply | Threaded
Open this post in threaded view
|

Re: pull request email notification

dsetrakyan
I actually like pull requests on the dev list. This tells the community
that something is available for review, no?

D.

On Mon, Aug 17, 2015 at 8:16 AM, Yakov Zhdanov <[hidden email]> wrote:

> Guys, why pull request notification goes to devlist, but not to
> [hidden email]
> ?
>
> --Yakov
>
Reply | Threaded
Open this post in threaded view
|

Re: pull request email notification

yzhdanov
There already is a way for doing this - move ticket to patch available.

Moreover, pull request may be created to trigger TC, which may be far from
final review version.

--Yakov

2015-08-17 18:31 GMT+03:00 Dmitriy Setrakyan <[hidden email]>:

> I actually like pull requests on the dev list. This tells the community
> that something is available for review, no?
>
> D.
>
> On Mon, Aug 17, 2015 at 8:16 AM, Yakov Zhdanov <[hidden email]>
> wrote:
>
> > Guys, why pull request notification goes to devlist, but not to
> > [hidden email]
> > ?
> >
> > --Yakov
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: pull request email notification

Raul Kripalani
In reply to this post by dsetrakyan
Pull requests are something you want the dev community to monitor
continuously. Contributions are welcome and important – and contributors
like getting a rapid response after the work they've put in voluntarily.

Not all committers monitor commits@ on an ongoing basis. Some committers
won't even subscribe to commits@ to begin with, and would prefer to rely on
their Git log history.

My preference is for PRs to continue being received on dev@.

Regards,

*Raúl Kripalani*
Apache Camel PMC Member & Committer | Enterprise Architect, Open Source
Integration specialist
http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani
http://blog.raulkr.net | twitter: @raulvk

On Mon, Aug 17, 2015 at 4:31 PM, Dmitriy Setrakyan <[hidden email]>
wrote:

> I actually like pull requests on the dev list. This tells the community
> that something is available for review, no?
>
> D.
>
> On Mon, Aug 17, 2015 at 8:16 AM, Yakov Zhdanov <[hidden email]>
> wrote:
>
> > Guys, why pull request notification goes to devlist, but not to
> > [hidden email]
> > ?
> >
> > --Yakov
> >
>