Merge pull request #48 from Onoyiza/test_os-win-mac

Test os win mac

Author: pgleeson

.github/workflows/ci.yml

@@ -19,12 +19,13 @@ jobs:
         extra_args: --hook-stage manual --all-files
 
   build:
-
-    runs-on: ubuntu-latest
+    name: Build for Python ${{ matrix.python-version }} on ${{ matrix.runs-on }}
+    runs-on: ${{ matrix.runs-on }}
     strategy:
       fail-fast: false
       matrix:
-        python-version: ["3.7", "3.8", "3.9", "3.10"]
+        python-version: [ "3.7", "3.8", "3.9", "3.10"]
+        runs-on: [ubuntu-latest, macos-latest, windows-latest]
 
     steps:
     - uses: actions/checkout@v2

docs/contributors.py

@@ -21,7 +21,17 @@
     url = per_info[i]
     print(url)
     data = requests.get(url=url)
-    empty_list.append(data.json())
+    requests_status = "unknown"
+    while (requests_status == "unknown") or (requests_status == "unsuccessful"):
+        if data.status_code == 200:
+            requests_status = "successful"
+            empty_list.append(data.json())
+        else:
+            # handle failure on requests to the url for mac os
+            requests_status = "unsuccessful"
+            print(f"Failed to get data from: {url}")
+            # make request again to get data from the url
+            data = requests.get(url=url)
 
 df1 = pd.DataFrame(empty_list)
 df1["name"] = df1["name"].fillna("")