Merge pull request #288 from tekktrik/dev/fix-daily-report

Fix daily report

Author: tekktrik

.gitignore

@@ -7,6 +7,7 @@ _build
 .bundles/*
 *.pyc
 .env
+.venv
 env.sh
 *.swp
 .libraries/*

adabot/lib/circuitpython_library_validators.py

@@ -984,7 +984,8 @@ def gather_insights(self, repo, insights, since, show_closed_metric=False):
                 if issue["state"] == "open":
                     if created > since:
                         insights["new_prs"] += 1
-                        insights["pr_authors"].add(pr_info["user"]["login"])
+                        if pr_info["user"]:
+                            insights["pr_authors"].add(pr_info["user"]["login"])
                     insights["active_prs"] += 1
                 else:
                     merged = datetime.datetime.strptime(
@@ -1031,7 +1032,10 @@ def gather_insights(self, repo, insights, since, show_closed_metric=False):
                         pr_reviews = gh_reqs.get(str(pr_info["url"]) + "/reviews")
                         if pr_reviews.ok:
                             for review in pr_reviews.json():
-                                if review["state"].lower() == "approved":
+                                if (
+                                    review["state"].lower() == "approved"
+                                    and review["user"]
+                                ):
                                     insights["pr_reviewers"].add(
                                         review["user"]["login"]
                                     )
@@ -1043,12 +1047,14 @@ def gather_insights(self, repo, insights, since, show_closed_metric=False):
                 if issue["state"] == "open":
                     if created > since:
                         insights["new_issues"] += 1
-                        insights["issue_authors"].add(issue_info["user"]["login"])
+                        if issue_info["user"]:
+                            insights["issue_authors"].add(issue_info["user"]["login"])
                     insights["active_issues"] += 1
 
                 else:
                     insights["closed_issues"] += 1
-                    insights["issue_closers"].add(issue_info["closed_by"]["login"])
+                    if issue_info["closed_by"]:
+                        insights["issue_closers"].add(issue_info["closed_by"]["login"])
 
         params = {"state": "open", "per_page": 100}
         issues = self.github_get_all_pages(