GitHub + notmuch scripts/hacks

2017-10-13 Thread William Casarin
Hey there, I've started collecting[1] a bunch of notmuch+GitHub hacks from some of the discussions @ id:87d16jtsdj@kyleam.com I've added the ability to comment on/approve/reject pull requests from a GitHub pull-request email[2]. It can also open a magit-log view of the PR from an email, as

Re: [PATCH v3 10/15] crypto: index encrypted parts when indexopts try_decrypt is set.

2017-10-13 Thread David Bremner
Daniel Kahn Gillmor writes: > > * properties prefixed with "index." are set during indexing and will be >cleared (and possibly re-set) upon reindexing > Fine. I think the model is that "applications" should take ownership of

Re: [PATCH v3 10/15] crypto: index encrypted parts when indexopts try_decrypt is set.

2017-10-13 Thread Daniel Kahn Gillmor
On Thu 2017-10-12 22:08:26 -0300, David Bremner wrote: > The second _notmuch_database_log will override the first > here. You can use _notmuch_database_log_append if you don't want to > clear the existing log (e.g. at least for the second _log > here). good catch, thanks! I'll include that in my

Re: Washing GitHub emails to include inline patch?

2017-10-13 Thread William Casarin
Kyle Meyer writes: > Looking at what I wrote again, I'd change DONT-FETCH to FORCE-FETCH and > then do something like > > (when (or force-fetch > (not (magit-ref-exists-p local-ref))) > (magit-call-git "fetch" "origin")) > > where local-ref is bound to