fix: Skip maintainer updates that conflict

We're using the github_id as a primary key for the NixMaintainer model
which prevents us from updating this field for a specific username and
we can't have duplicate usernames either since we have a unique
contstraint on them.

This "patches" #658 so ingestion works, until we figure out how to
handle #657.

Author: Erethon

src/shared/evaluation.py

@@ -196,18 +196,23 @@ def ingest_maintainers(
             # Duplicate...
             if m.github_id in seen:
                 continue
-
-            ms.append(
-                NixMaintainer.objects.update_or_create(
-                    defaults={
-                        "github": m.github,
-                        "email": m.email,
-                        "matrix": m.matrix,
-                        "name": m.name,
-                    },
-                    github_id=m.github_id,
+            try:
+                ms.append(
+                    NixMaintainer.objects.update_or_create(
+                        defaults={
+                            "github": m.github,
+                            "email": m.email,
+                            "matrix": m.matrix,
+                            "name": m.name,
+                        },
+                        github_id=m.github_id,
+                    )
                 )
-            )
+            except IntegrityError:
+                # Skip this maintainer until we decide how to handle #657
+                logger.debug(f"Skipping maintainer {m.github} due to username conflict")
+                continue
+
             seen.add(m.github_id)
 
         return [obj for obj, _ in ms]