[PATCH v3 1/5] parser: Remove unused parameter
Daniel Axtens
dja at axtens.net
Fri Dec 2 12:13:52 AEDT 2016
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 <dja at axtens.net>
Regards,
Daniel
More information about the Patchwork
mailing list