Merge pull request openSUSE#3041 from Vogtinator/review

check_source.py: Use correct function for getting maintainer of the devel pkg
DimStar77 · Jan 10, 2024 · 8329809 · 8329809
2 parents 6ab1d35 + aa1b3d6
commit 8329809
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/ReviewBot.py b/ReviewBot.py
@@ -896,7 +896,7 @@ def postoptparse(self):
         conf.get_config(override_apiurl=self.options.apiurl)
 
         if (self.options.osc_debug):
-            conf.config['debug'] = 1
+            conf.config['debug'] = True
 
         self.checker = self.setup_checker()
         if self.options.config:

diff --git a/check_source.py b/check_source.py
@@ -22,7 +22,7 @@
 from osclib.core import group_members
 from osclib.core import package_kind
 from osclib.core import create_add_role_request
-from osclib.core import maintainers_get
+from osclib.core import package_role_expand
 from osc.core import show_package_meta, show_project_meta
 from osc.core import get_request_list
 from urllib.error import HTTPError
@@ -300,7 +300,7 @@ def check_source_submission(
             devel_project, devel_package = devel_project_fallback(self.apiurl, target_project, target_package)
             if devel_project and devel_package:
                 submitter = self.request.creator
-                maintainers = set(maintainers_get(self.apiurl, devel_project, devel_package))
+                maintainers = set(package_role_expand(self.apiurl, devel_project, devel_package))
                 known_maintainer = False
                 if maintainers:
                     if submitter in maintainers: