From 73af7ebc783ed154dafdfa8e2ad9cfb0d490f504 Mon Sep 17 00:00:00 2001
From: Christopher Bohn <bohn@unl.edu>
Date: Thu, 20 Feb 2020 07:55:59 -0600
Subject: [PATCH] corrected equality/inequality error
---
prep-code-review-and-response.py | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/prep-code-review-and-response.py b/prep-code-review-and-response.py
index cd3b797..4012de2 100644
--- a/prep-code-review-and-response.py
+++ b/prep-code-review-and-response.py
@@ -72,7 +72,7 @@ def validate_forks():
course.get_user_groupsets()))[0].get_groups()
new_groups: List[CanvasUserGroup] = list(filter(lambda g: g.get_name() == new_groupset,
course.get_user_groupsets()))[0].get_groups()
- for student in sorted(students, key=lambda student:student.get_canvas_user().get_sortable_name()):
+ for student in sorted(students, key=lambda student: student.get_canvas_user().get_sortable_name()):
print(f'Validating {student.get_canvas_user().get_name()}')
old_group_name: str = list(filter(lambda g: student.get_canvas_user() in g.get_students(),
old_groups))[0].get_name().strip().replace(" ", "")
@@ -81,8 +81,9 @@ def validate_forks():
project_names: List[str] = list(map(lambda p: p.get_name(), projects))
if old_group_name in project_names:
project: GitlabProject = list(filter(lambda p: p.get_name() == old_group_name, projects))[0]
- if project.get_access_level() == 'private':
- print(f'\t{student.get_canvas_user().get_name()} has not set {old_group_name}.git to "private."')
+ if project.get_access_level() != 'private':
+ print(f'\t{student.get_canvas_user().get_name()} has not set {old_group_name}.git to "private" '
+ f'(access level is {project.get_access_level()}).')
users: List[GitlabUser] = project.get_all_users()
usernames: List[str] = list(map(lambda u: u.get_username(), users))
for TA in TAs:
--
GitLab