github pull request script

[Andrew Bartlett]

On Wed, 2015-10-21 at 16:40 +0200, Stefan Metzmacher wrote:

> BTW: can we have the attachements as "Content-Type: text/plain;
> charset=UTF-8;"
> instead of "application/text/x-diff".

Looking at what is generated, it looks like the application/ part
should be removed, compared with other patches I see on the list. 

I'll see what can be done about that. 

> Having an idication of fullname of the submitter in the From: header
> and/or
> a Reply-To: header would also be nice.

The tricky part is that this is not included in the notification.  We
would need to chase a URL in the notification to (for example):
 https://api.github.com/users/abartlet

However don't get too excited about the e-mail in there, most team
members have that turned off :-)(

> Having the mails gpg signed would be also great, but maybe a task for
> another day.

Patches to the script are welcome: 
https://github.com/abartlet/gh-mailinglist-notifications ;-)

Alterantly, the URL in the mail can be used to pull the branch over
https, if that helps.

Thanks,

Andrew Bartlett

-- 
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team         https://samba.org
Samba Development and Support, Catalyst IT   
https://catalyst.net.nz/services/samba





-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20151022/8e5053aa/signature.sig>