Use asottile/add-trailing-comma
diff --git a/pre_commit_hooks/check_added_large_files.py b/pre_commit_hooks/check_added_large_files.py
index 8d5f4c4..193b910 100644
--- a/pre_commit_hooks/check_added_large_files.py
+++ b/pre_commit_hooks/check_added_large_files.py
@@ -51,7 +51,7 @@
parser = argparse.ArgumentParser()
parser.add_argument(
'filenames', nargs='*',
- help='Filenames pre-commit believes are changed.'
+ help='Filenames pre-commit believes are changed.',
)
parser.add_argument(
'--maxkb', type=int, default=500,
diff --git a/pre_commit_hooks/check_ast.py b/pre_commit_hooks/check_ast.py
index 9809a3f..1090300 100644
--- a/pre_commit_hooks/check_ast.py
+++ b/pre_commit_hooks/check_ast.py
@@ -26,7 +26,7 @@
sys.version.partition(' ')[0],
))
print('\n{}'.format(
- ' ' + traceback.format_exc().replace('\n', '\n ')
+ ' ' + traceback.format_exc().replace('\n', '\n '),
))
retval = 1
return retval
diff --git a/pre_commit_hooks/check_case_conflict.py b/pre_commit_hooks/check_case_conflict.py
index 3d6cf74..0f78296 100644
--- a/pre_commit_hooks/check_case_conflict.py
+++ b/pre_commit_hooks/check_case_conflict.py
@@ -45,7 +45,7 @@
parser = argparse.ArgumentParser()
parser.add_argument(
'filenames', nargs='*',
- help='Filenames pre-commit believes are changed.'
+ help='Filenames pre-commit believes are changed.',
)
args = parser.parse_args(argv)
diff --git a/pre_commit_hooks/check_docstring_first.py b/pre_commit_hooks/check_docstring_first.py
index 8e658a1..f2f5b72 100644
--- a/pre_commit_hooks/check_docstring_first.py
+++ b/pre_commit_hooks/check_docstring_first.py
@@ -1,4 +1,5 @@
from __future__ import absolute_import
+from __future__ import print_function
from __future__ import unicode_literals
import argparse
@@ -30,7 +31,7 @@
'{}:{} Multiple module docstrings '
'(first docstring on line {}).'.format(
filename, sline, found_docstring_line,
- )
+ ),
)
return 1
elif found_code_line is not None:
@@ -38,7 +39,7 @@
'{}:{} Module docstring appears after code '
'(code seen on line {}).'.format(
filename, sline, found_code_line,
- )
+ ),
)
return 1
else:
diff --git a/pre_commit_hooks/check_merge_conflict.py b/pre_commit_hooks/check_merge_conflict.py
index 7d87efc..5035b6d 100644
--- a/pre_commit_hooks/check_merge_conflict.py
+++ b/pre_commit_hooks/check_merge_conflict.py
@@ -7,7 +7,7 @@
b'<<<<<<< ',
b'======= ',
b'=======\n',
- b'>>>>>>> '
+ b'>>>>>>> ',
]
WARNING_MSG = 'Merge conflict string "{0}" found in {1}:{2}'
diff --git a/pre_commit_hooks/debug_statement_hook.py b/pre_commit_hooks/debug_statement_hook.py
index 902198f..d76e6e6 100644
--- a/pre_commit_hooks/debug_statement_hook.py
+++ b/pre_commit_hooks/debug_statement_hook.py
@@ -29,7 +29,7 @@
def visit_ImportFrom(self, node):
if node.module in DEBUG_STATEMENTS:
self.debug_import_statements.append(
- DebugStatement(node.module, node.lineno, node.col_offset)
+ DebugStatement(node.module, node.lineno, node.col_offset),
)
@@ -52,7 +52,7 @@
debug_statement.line,
debug_statement.col,
debug_statement.name,
- )
+ ),
)
return 1
else:
diff --git a/pre_commit_hooks/detect_aws_credentials.py b/pre_commit_hooks/detect_aws_credentials.py
index 42758f0..b2afd71 100644
--- a/pre_commit_hooks/detect_aws_credentials.py
+++ b/pre_commit_hooks/detect_aws_credentials.py
@@ -12,7 +12,7 @@
files = set()
for env_var in (
'AWS_CONFIG_FILE', 'AWS_CREDENTIAL_FILE', 'AWS_SHARED_CREDENTIALS_FILE',
- 'BOTO_CONFIG'
+ 'BOTO_CONFIG',
):
if env_var in os.environ:
files.add(os.environ[env_var])
@@ -23,7 +23,7 @@
"""Extract AWS secrets from environment variables."""
keys = set()
for env_var in (
- 'AWS_SECRET_ACCESS_KEY', 'AWS_SECURITY_TOKEN', 'AWS_SESSION_TOKEN'
+ 'AWS_SECRET_ACCESS_KEY', 'AWS_SECURITY_TOKEN', 'AWS_SESSION_TOKEN',
):
if env_var in os.environ:
keys.add(os.environ[env_var])
@@ -50,7 +50,7 @@
for section in parser.sections():
for var in (
'aws_secret_access_key', 'aws_security_token',
- 'aws_session_token'
+ 'aws_session_token',
):
try:
keys.add(parser.get(section, var))
@@ -93,13 +93,13 @@
help=(
'Location of additional AWS credential files from which to get '
'secret keys from'
- )
+ ),
)
parser.add_argument(
'--allow-missing-credentials',
dest='allow_missing_credentials',
action='store_true',
- help='Allow hook to pass when no credentials are detected.'
+ help='Allow hook to pass when no credentials are detected.',
)
args = parser.parse_args(argv)
@@ -124,7 +124,7 @@
print(
'No AWS keys were found in the configured credential files and '
'environment variables.\nPlease ensure you have the correct '
- 'setting for --credentials-file'
+ 'setting for --credentials-file',
)
return 2
diff --git a/pre_commit_hooks/pretty_format_json.py b/pre_commit_hooks/pretty_format_json.py
index 5e04230..bb7a3d0 100644
--- a/pre_commit_hooks/pretty_format_json.py
+++ b/pre_commit_hooks/pretty_format_json.py
@@ -120,7 +120,7 @@
except simplejson.JSONDecodeError:
print(
"Input File {} is not a valid JSON, consider using check-json"
- .format(json_file)
+ .format(json_file),
)
return 1
diff --git a/pre_commit_hooks/tests_should_end_in_test.py b/pre_commit_hooks/tests_should_end_in_test.py
index 5f9bbac..9bea20d 100644
--- a/pre_commit_hooks/tests_should_end_in_test.py
+++ b/pre_commit_hooks/tests_should_end_in_test.py
@@ -11,7 +11,7 @@
parser.add_argument('filenames', nargs='*')
parser.add_argument(
'--django', default=False, action='store_true',
- help='Use Django-style test naming pattern (test*.py)'
+ help='Use Django-style test naming pattern (test*.py)',
)
args = parser.parse_args(argv)
@@ -27,8 +27,8 @@
retcode = 1
print(
'{} does not match pattern "{}"'.format(
- filename, test_name_pattern
- )
+ filename, test_name_pattern,
+ ),
)
return retcode
diff --git a/pre_commit_hooks/trailing_whitespace_fixer.py b/pre_commit_hooks/trailing_whitespace_fixer.py
index 1ae15a9..d44750d 100644
--- a/pre_commit_hooks/trailing_whitespace_fixer.py
+++ b/pre_commit_hooks/trailing_whitespace_fixer.py
@@ -36,7 +36,7 @@
const=[],
default=argparse.SUPPRESS,
dest='markdown_linebreak_ext',
- help='Do not preserve linebreak spaces in Markdown'
+ help='Do not preserve linebreak spaces in Markdown',
)
parser.add_argument(
'--markdown-linebreak-ext',
@@ -45,7 +45,7 @@
default=['md,markdown'],
metavar='*|EXT[,EXT,...]',
nargs='?',
- help='Markdown extensions (or *) for linebreak spaces'
+ help='Markdown extensions (or *) for linebreak spaces',
)
parser.add_argument('filenames', nargs='*', help='Filenames to fix')
args = parser.parse_args(argv)
@@ -69,7 +69,7 @@
parser.error(
"bad --markdown-linebreak-ext extension '{}' (has . / \\ :)\n"
" (probably filename; use '--markdown-linebreak-ext=EXT')"
- .format(ext)
+ .format(ext),
)
return_code = 0
