List notification after patch review and push
obnox at samba.org
Thu Feb 18 09:30:51 UTC 2016
On 2016-02-18 at 05:47 +0100, Ralph Boehme wrote:
> Hi folks!
> I would *love* to see an email after patches are revieved and pushed
> by others. Would save me a few minutes every time I get around to
> review a patch just to find it already pushed.
It happens now and then that I carve out some time to review a patch
that noone had responded to just to notice that it had already
been reviewed/pushed. Of course that is my personal problem and could
have been avoided by looking more thoroughly, checking git and
also the autobuild queue before reviewing. And it is not a real
waste of time to do additional code reviews, but sometimes it
just does not fit. So it is about asking for some kindness.
It would be kind to:
1. notify upon review completion/push.
2. for bigger patchsets where review takes longer, it might
sometimes even be nice to notify that one is in the process
Cheers - Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 198 bytes
Desc: not available
More information about the samba-technical