Looks good to me. Happily this should make the tests a little faster, which is always nice :)
> diff --git a/patchwork/tests/test_parser.py b/patchwork/tests/test_parser.py > index 96166ad..e4a379d 100644 > --- a/patchwork/tests/test_parser.py > +++ b/patchwork/tests/test_parser.py > @@ -62,7 +62,7 @@ def read_mail(filename, project=None): > mail = email.message_from_file(f) > if 'Message-Id' not in mail: > mail['Message-Id'] = make_msgid() > - if project is not None: > + if project: > mail['List-Id'] = project.listid > return mail This is just a 'clean up while I'm editing this file' change, right? Reviewed-by: Daniel Axtens <d...@axtens.net> Regards, Daniel _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork