Infrastructure/loopabull-tasks
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Declare the variable before using it and small style fix

Browse source 
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
This commit is contained in:
Pierre-Yves Chibon 2019-06-18 16:37:31 +02:00
parent 78e11431db
commit 7f2f2d5cdc
1 changed files with 2 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

5
playbooks/roles/flag_ci_pr/files/flag_ci_pr.py
View file

@ -44,6 +44,7 @@ def main(msg, pipeline_state='complete', seed='empty'):
print("Pipeline state is not 'complete' or 'running'.") print("Pipeline state is not 'complete' or 'running'.")
return return
commit_hash_text = ''
# test complete messages # test complete messages
if pipeline_state == 'complete': if pipeline_state == 'complete':
done_states = { done_states = {
@ -60,8 +61,6 @@ def main(msg, pipeline_state='complete', seed='empty'):
human_status = done_states[state]['human'] human_status = done_states[state]['human']
if 'commit_hash' in msg: if 'commit_hash' in msg:
commit_hash_text = ' for %s' % msg['commit_hash'][:8] commit_hash_text = ' for %s' % msg['commit_hash'][:8]
else:
commit_hash_text = ''
# test running messages # test running messages
elif pipeline_state == 'running': elif pipeline_state == 'running':
@ -78,7 +77,7 @@ def main(msg, pipeline_state='complete', seed='empty'):
data = { data = {
'username': 'Fedora CI', 'username': 'Fedora CI',
'status': status, 'status': status,
'comment': 'Package tests%s: %s' % (commit_hash_text,human_status), 'comment': 'Package tests%s: %s' % (commit_hash_text, human_status),
'url': msg['build_url'], 'url': msg['build_url'],
'uid': hashlib.md5(pr_id + seed).hexdigest() 'uid': hashlib.md5(pr_id + seed).hexdigest()
} }