CLN unaccounted for members variable in team job

AnubisLMS · Mar 18, 2024 · d3b2907 · d3b2907
1 parent 4cff204
commit d3b2907
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/api/anubis/github/team.py b/api/anubis/github/team.py
@@ -4,7 +4,7 @@
 def list_github_team_members(org: str, team: str) -> list[str]:
     return [
         user['login']
-        for user in github_rest(f"/orgs/{org}/teams/{team}/members")
+        for user in github_rest(f"/orgs/{org}/teams/{team}/members?per_page=100")
         if 'login' in user
     ]
 

diff --git a/api/anubis/jobs/daily_reaper.py b/api/anubis/jobs/daily_reaper.py
@@ -63,8 +63,12 @@ def reap_github_admin_teams():
 
             accounted_for_members.add(user.github_username)
 
+        unaccounted_for_members = set(members).difference(accounted_for_members)
+        logger.info(f'{members=}\n{accounted_for_members=}')
+        logger.info(f'members-accounted_for_members={unaccounted_for_members}')
+
         # Remove unaccounted for members
-        for github_username in set(members).difference(accounted_for_members):
+        for github_username in unaccounted_for_members:
             remote_github_team_member(course.github_org, course.github_ta_team_slug, github_username)