gitlab_mr_check 1.4.0__tar.gz → 1.4.2__tar.gz

{gitlab_mr_check-1.4.0 → gitlab_mr_check-1.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gitlab_mr_check
-Version: 1.4.0
+Version: 1.4.2
 Summary: GitLab Merge Request 4-Eyes Approval Audit Tool
 Author: Yorick Hoorneman
 Author-email: Yorick Hoorneman <yhoorneman@schubergphilis.com>
@@ -30,7 +30,7 @@ Description-Content-Type: text/markdown
 [![Conventional Commits](https://img.shields.io/badge/Conventional%20Commits-1.0.0-yellow.svg)](https://conventionalcommits.org)
 [![Changelog](https://img.shields.io/badge/changelog-Keep%20a%20Changelog%201.1.0-orange)](https://keepachangelog.com/en/1.1.0/)
 [![Build](https://img.shields.io/badge/build-passing-brightgreen)]()
-[![Coverage](https://img.shields.io/badge/coverage-34%25-red)](https://coverage.readthedocs.io/)
+[![Coverage](https://img.shields.io/badge/coverage-35%25-red)](https://coverage.readthedocs.io/)
 [![pyscn quality](https://img.shields.io/badge/pyscn-not%20rated-lightgrey)](https://pyscn.ludo-tech.org)
 
 GitLab Merge Request 4-Eyes Approval Audit Tool

{gitlab_mr_check-1.4.0 → gitlab_mr_check-1.4.2}/README.md

@@ -15,7 +15,7 @@
 [![Conventional Commits](https://img.shields.io/badge/Conventional%20Commits-1.0.0-yellow.svg)](https://conventionalcommits.org)
 [![Changelog](https://img.shields.io/badge/changelog-Keep%20a%20Changelog%201.1.0-orange)](https://keepachangelog.com/en/1.1.0/)
 [![Build](https://img.shields.io/badge/build-passing-brightgreen)]()
-[![Coverage](https://img.shields.io/badge/coverage-34%25-red)](https://coverage.readthedocs.io/)
+[![Coverage](https://img.shields.io/badge/coverage-35%25-red)](https://coverage.readthedocs.io/)
 [![pyscn quality](https://img.shields.io/badge/pyscn-not%20rated-lightgrey)](https://pyscn.ludo-tech.org)
 
 GitLab Merge Request 4-Eyes Approval Audit Tool

{gitlab_mr_check-1.4.0 → gitlab_mr_check-1.4.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "gitlab_mr_check"
-version = "1.4.0"
+version = "1.4.2"
 description = "GitLab Merge Request 4-Eyes Approval Audit Tool"
 readme = "README.md"
 authors = [
@@ -254,7 +254,7 @@ branch = true
 [tool.coverage.report]
 show_missing = true
 skip_covered = true
-fail_under = 34
+fail_under = 35
 exclude_lines = [
     "pragma: no cover",
     "if __name__ == .__main__.",

{gitlab_mr_check-1.4.0 → gitlab_mr_check-1.4.2}/src/gitlab_mr_check/gitlab_mr_check.py

@@ -143,9 +143,9 @@ def get_mrs_by_project(project: gitlab.v4.objects.Project, filters: list[Any]) -
     return [mr for mr in project.mergerequests.list(all=True) if all(f(mr) for f in filters)]
 
 
-def get_mrs_by_projects(projects: list[Any], filters: list[Any]) -> dict[str, list[Any]]:
-    """Return a mapping of project name to filtered MR list for each project."""
-    return {project.name: get_mrs_by_project(project, filters) for project in projects}
+def get_mrs_by_projects(projects: list[tuple[str, Any]], filters: list[Any]) -> dict[str, list[Any]]:
+    """Return a mapping of project name to filtered MR list for each (name, project) pair."""
+    return {name: get_mrs_by_project(project, filters) for name, project in projects}
 
 
 def evaluate_mrs_4eyes_per_project(project_mrs: dict[str, list[Any]]) -> list[ProjectMRAuditResult]:
@@ -190,20 +190,16 @@ def audit(url: str, token: str, config: Config) -> list[ProjectMRAuditResult]:
     LOGGER.info('Getting the projects under the groups')
     gl_projects = []
     for group in gl_groups:
-        group_projects = group.projects.list(all=True)
+        group_projects = group.projects.list(all=True, archived=False)
         LOGGER.info('  Group %s: %d project(s)', group.full_path, len(group_projects))
         for project in group_projects:
-            LOGGER.info('    Project: %s (archived: %s)', project.path_with_namespace, project.archived)
-            gl_projects.append(gl.projects.get(project.id))
-    LOGGER.info(
-        'Found %d project(s) (%d after excluding archived)',
-        len(gl_projects),
-        sum(1 for p in gl_projects if not p.archived),
-    )
+            LOGGER.info('    Project: %s', project.path_with_namespace)
+            gl_projects.append((project.path_with_namespace, gl.projects.get(project.id, lazy=True)))
+    LOGGER.info('Found %d project(s)', len(gl_projects))
 
     LOGGER.info('Getting the merge requests under the projects')
     gl_project_mrs = get_mrs_by_projects(
-        projects=[p for p in gl_projects if not p.archived],
+        projects=gl_projects,
         filters=[mr_is_merged, partial(mr_updated_in_years, years=config.gitlab.audit.years)],
     )