archive/actioner
Archived
1
Fork 0
Code Releases Packages Activity

Don't pass lists where it expects single items

Browse source 
This definitely worked in the past...
This commit is contained in:
Jake Howard 2019-08-31 16:03:32 +01:00
parent 2ef14e8d9e
commit ce78472619
Signed by: jake
GPG key ID: 57AFB45680EDD477
2 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
actioner/scheduler/todoist_assigned_issues.py
View file

@ -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)

6
actioner/scheduler/todoist_repo_prs.py
View file

@ -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))