Merge pull request #1812 from isuruf/cirun-all

Send all resources at once to cirun
conda-forge · Jan 8, 2024 · 1d54376 · 1d54376
2 parents 870b939 + 1c6b910
commit 1d54376
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 13 deletions.
diff --git a/conda_smithy/cirun_utils.py b/conda_smithy/cirun_utils.py
@@ -40,7 +40,7 @@ def enable_cirun_for_project(owner: str, repo: str) -> Dict[str, Any]:
 def add_repo_to_cirun_resource(
     owner: str,
     repo: str,
-    resource: str,
+    resources: List[str],
     teams: List,
     roles: List,
     users_from_json: Optional[str] = None,
@@ -52,7 +52,7 @@ def add_repo_to_cirun_resource(
     if cirun_policy_args and "pull_request" in cirun_policy_args:
         policy_args = {"pull_request": True}
     print(
-        f"Adding repo {owner}/{repo} to resource {resource} with policy_args: {policy_args}"
+        f"Adding repo {owner}/{repo} to resources {resources} with policy_args: {policy_args}"
     )
     gh = Github(gh_token())
     gh_owner = gh.get_user(owner)
@@ -67,7 +67,7 @@ def add_repo_to_cirun_resource(
     response = cirun.add_repo_to_resources(
         owner,
         repo,
-        resources=[resource],
+        resources=resources,
         policy_args=policy_args,
         teams=teams,
         roles=roles,

diff --git a/conda_smithy/cli.py b/conda_smithy/cli.py
@@ -432,16 +432,15 @@ def __call__(self, args):
                     f"Cirun Registration: resources to add to: {owner}/{repo}"
                 )
                 conda_smithy.cirun_utils.enable_cirun_for_project(owner, repo)
-                for resource in args.cirun_resources:
-                    conda_smithy.cirun_utils.add_repo_to_cirun_resource(
-                        owner,
-                        repo,
-                        resource,
-                        cirun_policy_args=args.cirun_policy_args,
-                        teams=args.cirun_teams,
-                        roles=args.cirun_roles,
-                        users_from_json=args.cirun_users_from_json,
-                    )
+                conda_smithy.cirun_utils.add_repo_to_cirun_resource(
+                    owner,
+                    repo,
+                    args.cirun_resources,
+                    cirun_policy_args=args.cirun_policy_args,
+                    teams=args.cirun_teams,
+                    roles=args.cirun_roles,
+                    users_from_json=args.cirun_users_from_json,
+                )
         else:
             print("Cirun registration disabled.")
 

diff --git a/news/cirun-resources.rst b/news/cirun-resources.rst
@@ -0,0 +1,23 @@
+**Added:**
+
+* <news item>
+
+**Changed:**
+
+* <news item>
+
+**Deprecated:**
+
+* <news item>
+
+**Removed:**
+
+* <news item>
+
+**Fixed:**
+
+* Code refactoring for cirun. (#1812)
+
+**Security:**
+
+* <news item>