Skip to content

Commit a83c3a5

Browse files
committed
pylint fixes
1 parent 73ff6af commit a83c3a5

File tree

2 files changed

+9
-12
lines changed

2 files changed

+9
-12
lines changed

web/client/codechecker_client/cmd/cmd.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1298,7 +1298,7 @@ def add_arguments_to_parser(parser):
12981298
formatter_class=argparse.ArgumentDefaultsHelpFormatter,
12991299
description="List the analysis runs available on the server.",
13001300
help="List the available analysis runs.")
1301-
1301+
13021302
runs.add_argument(
13031303
'--enabled-checkers',
13041304
action='store_true',
@@ -1544,4 +1544,4 @@ def add_arguments_to_parser(parser):
15441544
__add_common_arguments(permissions, needs_product_url=False)
15451545

15461546
# 'cmd' does not have a main() method in itself, as individual subcommands are
1547-
# handled later on separately.
1547+
# handled later on separately.

web/client/codechecker_client/cmd_line_client.py

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -654,19 +654,19 @@ def handle_list_runs(args):
654654
if result.analyzerName not in enabled_checkers:
655655
enabled_checkers[result.analyzerName] = set()
656656
enabled_checkers[result.analyzerName].add(result.checkerId)
657-
657+
658658
if args.output_format == 'plaintext':
659659
print("Enabled checkers:")
660-
for analyzer in enabled_checkers:
660+
for analyzer, checkers in enabled_checkers.items():
661661
print(analyzer + ":")
662-
for checker in enabled_checkers[analyzer]:
662+
for checker in checkers:
663663
print(" " + checker)
664664

665665
elif args.output_format == 'csv':
666666
separator = ','
667667
print("Analyzer" + separator + "Checker")
668-
for analyzer in enabled_checkers:
669-
for checker in enabled_checkers[analyzer]:
668+
for analyzer, checkers in enabled_checkers.items():
669+
for checker in checkers:
670670
print(analyzer + separator + checker)
671671

672672
elif args.output_format == 'json':
@@ -679,15 +679,12 @@ def handle_list_runs(args):
679679
header = ['Analyzer', 'Checker']
680680
rows = [
681681
(analyzer, checker)
682-
for analyzer in enabled_checkers
683-
for checker in enabled_checkers[analyzer]]
682+
for analyzer, checkers in enabled_checkers.items()
683+
for checker in checkers]
684684
print(twodim.to_str(args.output_format, header, rows))
685685

686686
else:
687687
LOG.error("Unsupported output format: %s", args.output_format)
688-
return 1
689-
690-
return 0
691688

692689
if args.output_format == 'json':
693690
# This json is different from the json format printed by the

0 commit comments

Comments
 (0)