From ce78472619f2a9a419867fb6d485872b1a2b3ae7 Mon Sep 17 00:00:00 2001 From: Jake Howard Date: Sat, 31 Aug 2019 16:03:32 +0100 Subject: [PATCH] Don't pass lists where it expects single items This definitely worked in the past... --- actioner/scheduler/todoist_assigned_issues.py | 4 ++-- actioner/scheduler/todoist_repo_prs.py | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/actioner/scheduler/todoist_assigned_issues.py b/actioner/scheduler/todoist_assigned_issues.py index 3c6888f..a4eabeb 100644 --- a/actioner/scheduler/todoist_assigned_issues.py +++ b/actioner/scheduler/todoist_assigned_issues.py @@ -36,7 +36,7 @@ def todoist_assigned_issues(): if existing_task_id and not me_assigned: logger.info("Deleting task for '{}'".format(issue.title)) - todoist.items.delete([existing_task_id]) + todoist.items.delete(existing_task_id) if not me_assigned: continue @@ -59,7 +59,7 @@ def todoist_assigned_issues(): if is_task_completed(existing_task): logger.info("Re-opening task '{}'".format(issue.title)) - todoist.items.uncomplete([existing_task_id]) + todoist.items.uncomplete(existing_task_id) existing_task.update( content=issue_to_task_name(issue), priority=get_status_for_issue(issue) diff --git a/actioner/scheduler/todoist_repo_prs.py b/actioner/scheduler/todoist_repo_prs.py index b247ea9..5cd6b32 100644 --- a/actioner/scheduler/todoist_repo_prs.py +++ b/actioner/scheduler/todoist_repo_prs.py @@ -37,10 +37,10 @@ def todoist_repo_prs(): if pr.merged and my_review and my_review.state == "APPROVED": if not is_task_completed(todoist.items.get_by_id(existing_task_id)): logger.info("Completing task to review '{}'".format(pr.title)) - todoist.items.complete([existing_task_id]) + todoist.items.complete(existing_task_id) else: logger.info("Deleting task to review '{}'".format(pr.title)) - todoist.items.delete([existing_task_id]) + todoist.items.delete(existing_task_id) elif pr.state == "open": my_review = get_my_review(me, pr) @@ -58,7 +58,7 @@ def todoist_repo_prs(): todoist.items.complete([existing_task_id]) elif task_completed: logger.info("Re-opening task to review '{}'".format(pr.title)) - todoist.items.uncomplete([existing_task_id]) + todoist.items.uncomplete(existing_task_id) continue elif my_review and my_review.commit_id != pr.head.sha: logger.info("Creating task to review '{}'".format(pr.title))