diff --git a/api/gitlab_classes.py b/api/gitlab_classes.py index 3f2e584975ea973cb067feb4971a93e432cf97d2..804e7e7c541d2f2ca6988a0b269ebd4abb9c840b 100644 --- a/api/gitlab_classes.py +++ b/api/gitlab_classes.py @@ -348,7 +348,7 @@ class GitlabProject: def get_visibility(self) -> str: """ - :return: 'private', etc. + :return: 'private', 'public', or 'internal'. """ return self.git_project.visibility @@ -433,14 +433,6 @@ class GitlabProject: return commits - # repository_access_level, - def get_access_level(self) -> str: - """ - :return: The access level of the repository ('private' is private; 'enabled' is public) - """ - return self.git_project.repository_access_level - - def get_labels(self) -> Set[str]: """ :return: set of label names @@ -464,6 +456,7 @@ class GitlabProject: # repo_branches, labels, events, members # empty_repo # archived + # repository_access_level, # resolve_outdated_diff_discussions # container_registry_enabled # issues_enabled diff --git a/prep-code-review-and-response.py b/prep-code-review-and-response.py index b1820b512680a42587ffedc5ec8391eb41bc51c3..7f90b0dcfe9df88c769386762d668ec106e264b2 100644 --- a/prep-code-review-and-response.py +++ b/prep-code-review-and-response.py @@ -81,10 +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] - # # NOT WORKING AS EXPECTED - # 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()}).') + if project.get_visibility() != '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_visibility()}).') users: List[GitlabUser] = project.get_all_users() usernames: List[str] = list(map(lambda u: u.get_username(), users)) for TA in TAs: