ISSUE-4: Boyscouting

Author: AFulgens

.pre-commit-config.yaml

@@ -2,7 +2,7 @@ default_install_hook_types: [ commit-msg, pre-commit, prepare-commit-msg ]
 default_stages: [ pre-commit ]
 repos:
 -   repo: https://github.com/ShellMagick/shellmagick-commit-hooks
-    rev: v24.03.1
+    rev: v24.06
     hooks:
     -   id: commiticketing
         args: [ -t= ]
@@ -19,7 +19,7 @@ repos:
     -   id: fix-ligatures
     -   id: forbid-bidi-controls
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.5.0
+    rev: v4.6.0
     hooks:
     -   id: check-added-large-files
         args: [ --maxkb=16 ]
@@ -47,7 +47,7 @@ repos:
     hooks:
     -   id: setup-cfg-fmt
 -   repo: https://github.com/asottile/reorder-python-imports
-    rev: v3.12.0
+    rev: v3.13.0
     hooks:
     -   id: reorder-python-imports
         args: [ --py39-plus, --add-import, 'from __future__ import annotations' ]
@@ -56,7 +56,7 @@ repos:
     hooks:
     -   id: add-trailing-comma
 -   repo: https://github.com/asottile/pyupgrade
-    rev: v3.15.1
+    rev: v3.16.0
     hooks:
     -   id: pyupgrade
         args: [ --py39-plus ]
@@ -66,6 +66,6 @@ repos:
 #    hooks:
 #    -   id: autopep8
 -   repo: https://github.com/PyCQA/flake8
-    rev: 7.0.0
+    rev: 7.1.0
     hooks:
     -   id: flake8

hooks/commiticketing.py

@@ -75,20 +75,20 @@ def main(argv: Sequence[str] | None = None) -> int:
     parser.add_argument(
         '-e', '--exclude-long-prefix', action='append',
         help='Do not add long prefix for these branches'
-             '(by default: feature, user, backup), '
-             'may be specified multiple times',
+             ' (by default: feature, user, backup),'
+             ' may be specified multiple times',
     )
     parser.add_argument(
         '-t', '--two-level', action='append',
         help='Get prefix from second level,'
-             'if branch starts with this (by default: user, backup), '
-             'may be specified multiple times',
+             ' if branch starts with this (by default: user, backup),'
+             ' may be specified multiple times',
     )
     parser.add_argument(
         '-b', '--branch', action='append',
-        help='Branch to process with this hook '
-             '(by default: feature, bugfix, hotfix), '
-             'may be specified multiple times',
+        help='Branch to process with this hook'
+             ' (by default: feature, bugfix, hotfix),'
+             ' may be specified multiple times',
     )
     args = parser.parse_args(argv)
 

hooks/no_todos.py

@@ -10,15 +10,15 @@ def main(argv: Sequence[str] | None = None) -> int:
     parser.add_argument('filenames', nargs='*', help='Filenames to check')
     parser.add_argument(
         '-t', '--todo-tag', action='append',
-        help='TODO tag to disallow as file content '
-             '(by default: TODO, FIXME, XXX), '
-             'may be specified multiple times',
+        help='TODO tag to disallow as file content'
+             ' (by default: TODO, FIXME, XXX),'
+             ' may be specified multiple times',
     )
     parser.add_argument(
         '-e', '--except-in', action='append', default=[],
         help='Except in these files '
-             '(i.e., a file with this name is allowed to contain the tags), '
-             'may be specified multiple times',
+             '(i.e., a file with this name is allowed to contain the tags),'
+             ' may be specified multiple times',
     )
     args = parser.parse_args(argv)
 

mutmut_config.py

@@ -5,7 +5,7 @@ def pre_mutation(context):
     line = context.current_source_line.strip()
     if line.startswith('parser.'):
         context.skip = True
-    if line == "if __name__ == '__main__':":
+    elif line == "if __name__ == '__main__':":
         context.skip = True
-    if line.strip().startswith("help='"):
+    elif line.strip().startswith("help='"):
         context.skip = True

setup.cfg

@@ -40,6 +40,7 @@ check_untyped_defs = true
 disallow_any_generics = true
 disallow_incomplete_defs = true
 disallow_untyped_defs = true
+show_error_codes = true
 warn_redundant_casts = true
 warn_unused_ignores = true