Stephen Finucane <step...@that.guru> writes:

> I thought there was a bug. Turns out I was just using the wrong list-id.
> Make this clearer.
>
> Signed-off-by: Stephen Finucane <step...@that.guru>
> ---
>  patchwork/parser.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/patchwork/parser.py b/patchwork/parser.py
> index 1568bc44..37e4365c 100644
> --- a/patchwork/parser.py
> +++ b/patchwork/parser.py
> @@ -156,7 +156,7 @@ def find_project_by_id(list_id):
>      try:
>          project = Project.objects.get(listid=list_id)
>      except Project.DoesNotExist:
> -        pass
> +        logger.error("'%s' if not a valid project list-id", list_id)
                              ^ maybe this should be 'is' ?

Also maybe it should be an info log?  Afterall, there's no error here
from the application.

>      return project

_______________________________________________
Patchwork mailing list
Patchwork@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/patchwork

Reply via email to