diff --git a/pre_commit_hooks/pretty_format_json.py b/pre_commit_hooks/pretty_format_json.py index f3afe98..c15fbda 100644 --- a/pre_commit_hooks/pretty_format_json.py +++ b/pre_commit_hooks/pretty_format_json.py @@ -2,9 +2,8 @@ from __future__ import annotations import argparse import json -import sys import re - +import sys from difflib import unified_diff from typing import Mapping from typing import Sequence @@ -129,7 +128,7 @@ def main(argv: Sequence[str] | None = None) -> int: pretty_contents = _get_pretty_format( contents, args.indent, ensure_ascii=not args.no_ensure_ascii, sort_keys=not args.no_sort_keys, top_keys=args.top_keys, - empty_object_with_newline=args.empty_object_with_newline + empty_object_with_newline=args.empty_object_with_newline, ) except ValueError: print( @@ -148,7 +147,6 @@ def main(argv: Sequence[str] | None = None) -> int: diff_output = get_diff(contents, pretty_contents, json_file) sys.stdout.buffer.write(diff_output.encode()) - return status diff --git a/tests/pretty_format_json_test.py b/tests/pretty_format_json_test.py index 4413d17..bd7dab9 100644 --- a/tests/pretty_format_json_test.py +++ b/tests/pretty_format_json_test.py @@ -1,10 +1,10 @@ from __future__ import annotations +import filecmp import os import shutil import pytest -import filecmp from pre_commit_hooks.pretty_format_json import main from pre_commit_hooks.pretty_format_json import parse_num_to_int @@ -142,8 +142,8 @@ def test_diffing_output(capsys): def test_empty_object_with_newline(tmpdir): # same line objects shoud trigger with --empty-object-with-newline switch - sameline = get_resource_path("empty_object_json_sameline.json") - ret = main(["--empty-object-with-newline", str(sameline)]) + sameline = get_resource_path('empty_object_json_sameline.json') + ret = main(['--empty-object-with-newline', str(sameline)]) assert ret == 1 # a template to be compared against.