Skip to content

Commit

Permalink
cli: Drop --json as there's no other output format anyway
Browse files Browse the repository at this point in the history
  • Loading branch information
barthalion committed Oct 6, 2023
1 parent 2c946bf commit 93d267a
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions flatpak_builder_lint/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,6 @@ def main() -> int:
parser = argparse.ArgumentParser(
description="A linter for Flatpak builds and flatpak-builder manifests"
)
parser.add_argument("--json", help="output in JSON format", action="store_true")
parser.add_argument(
"--version", action="version", version=f"flatpak-builder-lint {__version__}"
)
Expand Down Expand Up @@ -133,13 +132,7 @@ def main() -> int:
if "errors" in results:
exit_code = 1

if args.json:
output = json.dumps(results, indent=4)
else:
# we default to JSON output anyway as it's nicely formatted
# TODO: make the output more human readable
output = json.dumps(results, indent=4)

output = json.dumps(results, indent=4)
print(output)

if args.exceptions and not args.json:

This comment has been minimized.

Copy link
@muelli

muelli Oct 9, 2023

@barthalion here, args.json is still referenced.

This comment has been minimized.

Copy link
@hfiguiere

hfiguiere Oct 9, 2023

Contributor

we are aware. See #183 and #184

Expand Down

0 comments on commit 93d267a

Please sign in to comment.