From 9086279b5d6cd6603024449a6f14412e07f9ec73 Mon Sep 17 00:00:00 2001 From: Riccardo Magliocchetti Date: Wed, 23 May 2018 15:34:23 +0200 Subject: [PATCH] oauth/services: correct error handling in paginate the else code was dead code as the code in the try block already returned. Instead return an empty list when it is needed. --- readthedocs/oauth/services/base.py | 1 - 1 file changed, 1 deletion(-) diff --git a/readthedocs/oauth/services/base.py b/readthedocs/oauth/services/base.py index ee1a6b480ee..3b5b60b02ba 100644 --- a/readthedocs/oauth/services/base.py +++ b/readthedocs/oauth/services/base.py @@ -153,7 +153,6 @@ def paginate(self, url, **kwargs): url, debug_data, ) - else: return [] def sync(self):