From 3791c246c184816d6dfc72cb4e69b32ee652bfa1 Mon Sep 17 00:00:00 2001 From: Gwyn Ciesla Date: Jul 24 2019 13:17:15 +0000 Subject: [PATCH 1/5] On auto-approve, only prompt on anomalies. .gitignore addition Bump version. --- diff --git a/.gitignore b/.gitignore index 84b4f8c..34643c6 100644 --- a/.gitignore +++ b/.gitignore @@ -10,3 +10,4 @@ dist/ .coverage .vscode/ .pytest_cache/ +*~ diff --git a/fedscm_admin/utils.py b/fedscm_admin/utils.py index 9459c30..555fbb2 100644 --- a/fedscm_admin/utils.py +++ b/fedscm_admin/utils.py @@ -419,7 +419,10 @@ def prompt_for_new_repo(issue_json, issue_body_json, force=False, 'manually verify it.') click.secho(msg, fg='yellow') - action = prompt_for_ticket_action() + if auto_approve == False: + action = prompt_for_ticket_action() + else: + action = 'approve' if action == 'approve': # Create the PDC SLA entry @@ -468,7 +471,7 @@ def prompt_for_new_repo(issue_json, issue_body_json, force=False, 'You may commit to the branch "{1}" in about ' '10 minutes.'.format(dist_git_url, branch_name)) - comment_and_close_ticket(issue_id, bug_id, new_repo_comment) + comment_and_close_ticket(issue_id, bug_id, new_repo_comment, prompt_for_comment=False) elif action == 'deny': comment_body = click.prompt( 'Please enter a comment explaining the denial') @@ -561,7 +564,10 @@ def prompt_for_new_branch(issue_json, issue_body_json, auto_approve=False): click.echo(' Git Branch: {0}'.format(bool_to_word( create_git_branch))) - action = prompt_for_ticket_action() + if auto_approve == False: + action = prompt_for_ticket_action() + else: + action = 'approve' if action == 'approve': pagure_url = get_config_item(CONFIG, 'pagure_dist_git_url') @@ -591,7 +597,7 @@ def prompt_for_new_branch(issue_json, issue_body_json, auto_approve=False): 'the request, but in about 10 minutes, you may create the ' 'branch in Pagure using git.') comment_and_close_ticket(issue_id, bug_id, new_branch_comment, - prompt_for_comment=not auto_approved) + prompt_for_comment=False) elif action == 'deny': comment_body = click.prompt( 'Please enter a comment explaining the denial') From 176c3eb589bdc257ad3daf1b0c806897627d9709 Mon Sep 17 00:00:00 2001 From: Gwyn Ciesla Date: Jul 24 2019 13:17:58 +0000 Subject: [PATCH 2/5] On auto-approve, only prompt on anomalies. .gitignore addition Bump version. --- diff --git a/setup.py b/setup.py index a0fc11f..c44b1f0 100644 --- a/setup.py +++ b/setup.py @@ -20,5 +20,5 @@ setup( packages=['fedscm_admin'], package_dir={'fedscm_admin': 'fedscm_admin'}, url='https://pagure.io/fedscm_admin', - version='1.0.9', + version='1.0.10', ) From d728f9e558332f33e318b9e4440e9dcbcf2dc980 Mon Sep 17 00:00:00 2001 From: Gwyn Ciesla Date: Jul 24 2019 13:18:15 +0000 Subject: [PATCH 3/5] Bump version. --- diff --git a/setup.py b/setup.py index c44b1f0..a0fc11f 100644 --- a/setup.py +++ b/setup.py @@ -20,5 +20,5 @@ setup( packages=['fedscm_admin'], package_dir={'fedscm_admin': 'fedscm_admin'}, url='https://pagure.io/fedscm_admin', - version='1.0.10', + version='1.0.9', ) From 013ef1ddbaa7384da06b1d631379fea7844d03b5 Mon Sep 17 00:00:00 2001 From: Gwyn Ciesla Date: Jul 24 2019 13:20:10 +0000 Subject: [PATCH 5/5] Bump version. --- diff --git a/setup.py b/setup.py index a0fc11f..c44b1f0 100644 --- a/setup.py +++ b/setup.py @@ -20,5 +20,5 @@ setup( packages=['fedscm_admin'], package_dir={'fedscm_admin': 'fedscm_admin'}, url='https://pagure.io/fedscm_admin', - version='1.0.9', + version='1.0.10', )