commit 0f7b5e0c4f9f0a7516ae84be9df898e02fd34833
author Anthony Sottile <asottile@umich.edu> Sat Oct 12 13:53:50 2019 -0700
committer GitHub <noreply@github.com> Sat Oct 12 13:53:50 2019 -0700
tree d8e2d2fcde41d3d4d0620b775c7944a076f34c9f
parent 49f8b48f2e43cbdb8e025751e8e202cee6145b14
parent 86a010b750873ae00429f333c6e83ba7f8a7e29e

Merge pull request #419 from pre-commit/extra_output

Remove unnecessary double printing of filename

pre_commit_hooks/pretty_format_json.py

diff --git a/pre_commit_hooks/pretty_format_json.py b/pre_commit_hooks/pretty_format_json.py
index cbc1b19..7d515f4 100644
--- a/pre_commit_hooks/pretty_format_json.py
+++ b/pre_commit_hooks/pretty_format_json.py
@@ -120,12 +120,6 @@
             )
 
             if contents != pretty_contents:
-                print(
-                    'File {} is not pretty-formatted'.format(json_file),
-                    file=sys.stderr,
-                )
-                sys.stderr.flush()
-
                 if args.autofix:
                     _autofix(json_file, pretty_contents)
                 else:

tests/pretty_format_json_test.py

diff --git a/tests/pretty_format_json_test.py b/tests/pretty_format_json_test.py
index 6cfe772..b42e504 100644
--- a/tests/pretty_format_json_test.py
+++ b/tests/pretty_format_json_test.py
@@ -131,11 +131,9 @@
 +  "foo": "bar"
  }}
 '''.format(a, b)
-    expected_err = 'File {} is not pretty-formatted\n'.format(resource_path)
-
     actual_retval = main([resource_path])
     actual_out, actual_err = capsys.readouterr()
 
     assert actual_retval == expected_retval
     assert actual_out == expected_out
-    assert actual_err == expected_err
+    assert actual_err == ''