On Fri, 2016-12-02 at 13:49 +1100, Daniel Axtens wrote: > > - delegate = find_delegate(mail) > > + delegate = find_delegate_by_header(mail) > > if not delegate and diff: > > filenames = find_filenames(diff) > > - delegate = auto_delegate(project, filenames) > > + delegate = find_delegate_by_filename(project, > > filenames) > > It's not 100% clear from the context: are we using multiple > filenames, > or just one filename? Your argument and function name don't match up.
Good call - it's multiple filenames. I've applied this as is to fix Tom's issue, but I'll address this in a follow-up patch. Stephen > Apart from that it looks good to me. > > Regards, > Daniel _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork
