Improved get top contrib script

.github/workflows/build.yml

@@ -21,14 +21,14 @@ jobs:
       with:
         node-version: "${{ steps.nvm.outputs.NVMRC }}"
     - name: Cache npm
-      uses: actions/cache@v1
+      uses: actions/cache@v4
       with:
         path: ~/.npm
         key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
         restore-keys: |
           ${{ runner.os }}-node-
     - name: Cache pip
-      uses: actions/cache@v1
+      uses: actions/cache@v4
       with:
         path: ~/.cache/pip
         key: ${{ runner.os }}-pip-${{ hashFiles('**/Pipfile.lock') }}

content-repo/gen_top_contrib.py

@@ -131,25 +131,26 @@ def github_pagination_prs(url: str, params: dict, res) -> list:
         params (dict): params for the request
         res: the response from the http_request
 
-    Returns: prs (dict)
+    Returns: prs (list)
     """
     prs = []
     if link := res.headers.get('Link'):
         links = link.split(',')
         last_page = links[-1]
         last_page_link = last_page[last_page.find("<") + 1:last_page.find(">")]
-        try:
-            last_pr_number = PAGE_NUMBER_REGEX.search(last_page_link)[0]
-        except IndexError:
-            print(f"Error: Couldn't find the last page number. \n Last page link - {last_page_link}")
-            return
-
-        while params['page'] < int(last_pr_number):
-            params['page'] = params['page'] + 1
-            response = search_session.request('GET', url, params=params, headers=HEADERS, verify=VERIFY)
-            response.raise_for_status()
-            next_page_prs = response.json().get('items', [])
-            prs.extend(next_page_prs)
+        print(f"{last_page_link}: {last_page}")
+
+        match = PAGE_NUMBER_REGEX.search(last_page_link)
+        if match:
+            last_pr_number = int(match.group(1))
+            while params['page'] < last_pr_number:
+                params['page'] = params['page'] + 1
+                response = search_session.request('GET', url, params=params, headers=HEADERS, verify=VERIFY)
+                response.raise_for_status()
+                next_page_prs = response.json().get('items', [])
+                prs.extend(next_page_prs)
+        else:
+            print(f"Warning: Couldn't find the last page number. Last page link - {last_page_link}")
 
     return prs