Skip to content

Commit 6fce70b

Browse files
author
Caspar van Leeuwen
committed
Fix that the test result doesnt overwrite the finished result
1 parent 010ef0b commit 6fce70b

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

tasks/build.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1333,13 +1333,17 @@ def request_bot_build_issue_comments(repo_name, pr_number):
13331333
result = ':shrug: UNKNOWN'
13341334
else:
13351335
result = row['comment']
1336-
else:
1336+
elif row['job_status'] == 'submitted' or row['job_status'] == 'received' or row['job_status'] == 'running':
13371337
# Make sure that if the job is not finished yet, we also put something useful in these fields
13381338
# It is useful to know a job is submitted, running, etc
13391339
date = row['date']
13401340
status = row['job status']
13411341
url = comment['html_url']
13421342
result = row['comment']
1343+
else:
1344+
# Don't do anything for the test line for now - we might add an extra entry to the status
1345+
# table later to reflect the test result
1346+
continue
13431347

13441348
# Add all entries to status_table. We do this at the end of this loop so that the operation is
13451349
# more or less 'atomic', i.e. all vectors in the status_table dict have the same length

0 commit comments

Comments
 (0)