feat: create a should_skip function

Author: paolomainardi

pr_agent/servers/gitlab_webhook.py

@@ -75,6 +75,12 @@ async def _perform_commands_gitlab(commands_conf: str, agent: PRAgent, api_url:
             get_logger().error(f"Failed to perform command {command}: {e}")
 
 
+def should_skip_mr(draft: bool, mr_url: str):
+    skip_draft_mr = get_settings().get("GITLAB.SKIP_DRAFT_MR", False)
+    if draft and skip_draft_mr:
+        get_logger().info(f"Skipping draft MR: {mr_url}")
+        return True
+    return False
 
 @router.post("/webhook")
 async def gitlab_webhook(background_tasks: BackgroundTasks, request: Request):
@@ -130,8 +136,7 @@ async def inner(data: dict):
             url = data['object_attributes'].get('url')
             draft = data['object_attributes'].get('draft')
 
-            if draft and skip_draft_mr:
-                get_logger().info(f"Skipping draft MR: {url}")
+            if should_skip_mr(draft, url):
                 return JSONResponse(status_code=status.HTTP_200_OK, content=jsonable_encoder({"message": "success"}))
 
             get_logger().info(f"New merge request: {url}")
@@ -143,8 +148,7 @@ async def inner(data: dict):
                 draft = mr.get('draft')
 
                 get_logger().info(f"A comment has been added to a merge request: {url}")
-                if draft and skip_draft_mr:
-                    get_logger().info(f"Skipping draft MR: {url}")
+                if should_skip_mr(draft, url):
                     return JSONResponse(status_code=status.HTTP_200_OK, content=jsonable_encoder({"message": "success"}))
 
                 body = data.get('object_attributes', {}).get('note')