Skip to content

Commit ab6ca23

Browse files
committed
pwclient: Rename 'checks' -> 'check-list'
Rename this function to ensure all check-related commands have a 'check-' prefix. Signed-off-by: Stephen Finucane <stephen.finucane@intel.com> Tested-by: Andy Doan <andy.doan@linaro.org>
1 parent b78ec3f commit ab6ca23

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

patchwork/bin/pwclient

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@ def action_projects(rpc):
236236
project['name']))
237237

238238

239-
def action_checks(rpc):
239+
def action_check_list(rpc):
240240
checks = rpc.check_list()
241241
for check in checks:
242242
print("%d (for '%s')" % (check['id'], check['patch']))
@@ -474,12 +474,12 @@ def main():
474474
help='''List all projects'''
475475
)
476476
projects_parser.set_defaults(subcmd='projects')
477-
checks_parser = subparsers.add_parser(
478-
'checks',
477+
check_list_parser = subparsers.add_parser(
478+
'check-list',
479479
add_help=False,
480-
help='''Show list of patch checks'''
480+
help='''List all checks'''
481481
)
482-
checks_parser.set_defaults(subcmd='checks')
482+
check_list_parser.set_defaults(subcmd='check_list')
483483
check_create_parser = subparsers.add_parser(
484484
'check-create', parents=[hash_parser], conflict_handler='resolve',
485485
help='Add a check to a patch')
@@ -708,8 +708,6 @@ def main():
708708
elif action.startswith('project'):
709709
action_projects(rpc)
710710

711-
elif action.startswith('checks'):
712-
action_checks(rpc)
713711

714712
elif action.startswith('state'):
715713
action_states(rpc)
@@ -766,6 +764,9 @@ def main():
766764
archived=archived_str, commit=commit_str
767765
)
768766

767+
elif action == 'check_list':
768+
action_check_list(rpc)
769+
769770
elif action == 'check_create':
770771
for patch_id in non_empty(h, patch_ids):
771772
action_check_create(

0 commit comments

Comments
 (0)