On Fri 2019-11-01 20:37:34 -0700, Sean Whitton wrote:
> I've applied your first patch to my master.

hm, i don't see this merge when i "git remote update origin" (where
"origin" is https://git.spwhitton.name/mailscripts).  Could you push?

> The second patch is quite hard for me to review because your
> refactoring of the basic structure of the script is in the same commit
> as adding the new feature.  Could you split that into two or more
> commits, please?

yes, i'll do that and send the updated patches here.

thanks for the prompt review.

     --dkg

Attachment: signature.asc
Description: PGP signature

Reply via email to