Skip to content

Commit

Permalink
Merge 83f03c7 into 67557a7
Browse files Browse the repository at this point in the history
  • Loading branch information
timgates42 committed May 18, 2021
2 parents 67557a7 + 83f03c7 commit 64de935
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
9 changes: 5 additions & 4 deletions app/meticulous/_multiworker.py
Expand Up @@ -313,7 +313,8 @@ def multiworker_core(interaction, target):
controller = Controller(
handlers=handlers, input_queue=input_queue, threadpool=threadpool, target=target
)
for task in workload:
controller.add(task)
result = controller.run(interaction)
set_json_value(key, result)
with controller:
for task in workload:
controller.add(task)
result = controller.run(interaction)
set_json_value(key, result)
5 changes: 3 additions & 2 deletions app/meticulous/_submit.py
Expand Up @@ -32,7 +32,7 @@ def get_note(kind, pr_url=None):
f"https://github.com/timgates42/meticulous"
f"/blob/master/docs/NOTE.md"
)
if pr_url is None:
if kind != "issue" or pr_url is None:
return header
api = get_api()
user_org = api.get_user().login
Expand All @@ -42,7 +42,8 @@ def get_note(kind, pr_url=None):
To avoid wasting CI processing resources a branch with the fix has been
prepared but a pull request has not yet been created. A pull request fixing
the issue can be prepared from the link below, feel free to create it or
request @{user_org} create the PR.
request @{user_org} create the PR. Alternatively if the fix is undesired please
close the issue with a small comment about the reasoning.
{pr_url}
Expand Down

0 comments on commit 64de935

Please sign in to comment.