Handle search API error

saadmk11 · Sep 3, 2020 · f444f84 · f444f84
1 parent 6ebed89
commit f444f84
Showing 1 changed file with 21 additions and 11 deletions.
diff --git a/scripts/changelog-ci.py b/scripts/changelog-ci.py
@@ -89,17 +89,27 @@ def _get_pull_requests_after_last_release(self):
 
         response = requests.get(url)
 
-        response_data = response.json()
-
-        if response_data['total_count'] > 0:
+        if response.status_code == 200:
+            response_data = response.json()
 
-            for item in response_data['items']:
-                data = {
-                    'title': item['title'],
-                    'number': item['number'],
-                    'url': item['html_url']
-                }
-                items.append(data)
+            if response_data['total_count'] > 0:
+                for item in response_data['items']:
+                    data = {
+                        'title': item['title'],
+                        'number': item['number'],
+                        'url': item['html_url']
+                    }
+                    items.append(data)
+            else:
+                logger.warning(
+                    'There was no pull request made on %s after last release.',
+                    self.repository
+                )
+        else:
+            logger.error(
+                'GitHub API returned error response for %s, status code: %s',
+                self.repository, response.status_code
+            )
 
         return items
 
@@ -116,8 +126,8 @@ def write_changelog(self):
 
         items = self._get_pull_requests_after_last_release()
 
+        # exit the function if there is not pull request found
         if not items:
-            print('There was no pull request made after last release.')
             return
 
         file_mode = self._get_file_mode()