diff --git a/prep-code-review-and-response.py b/prep-code-review-and-response.py index cd3b79776aecb6feade71e2709e8592e06995348..4012de2b3370748a5cd5c7fe4ccae59f4ebe75ed 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: