diff --git a/README.md b/README.md
@@ -110,6 +110,8 @@ To typecheck, run: `mypy -p git`

To test, run: `pytest`

Configuration for flake8 is in the ./.flake8 file.

Configurations for mypy, pytest and coverage.py are in ./pyproject.toml.

diff --git a/git/config.py b/git/config.py
@@ -248,7 +248,6 @@ def items_all(self) -> List[Tuple[str, List[_T]]]:


def get_config_path(config_level: Lit_config_levels) -> str:
-
    # we do not support an absolute path of the gitconfig on windows ,
    # use the global config instead
    if is_win and config_level == "system":
@@ -265,8 +264,8 @@ def get_config_path(config_level: Lit_config_levels) -> str:
        raise ValueError("No repo to get repository configuration from. Use Repo._get_config_path")
    else:
        # Should not reach here. Will raise ValueError if does. Static typing will warn missing elifs
-        assert_never(
-            config_level,  # type: ignore[unreachable]
            ValueError(f"Invalid configuration level: {config_level!r}"),
        )

@@ -655,7 +654,7 @@ def write_section(name: str, section_dict: _OMD) -> None:

            values: Sequence[str]  # runtime only gets str in tests, but should be whatever _OMD stores
            v: str
-            for (key, values) in section_dict.items_all():
                if key == "__name__":
                    continue


diff --git a/git/diff.py b/git/diff.py
@@ -145,7 +145,7 @@ def diff(
        args.append("--full-index")  # get full index paths, not only filenames

        # remove default '-M' arg (check for renames) if user is overriding it
-        if not any(x in kwargs for x in ('find_renames', 'no_renames', 'M')):
            args.append("-M")

        if create_patch:
@@ -338,7 +338,6 @@ def __init__(
        change_type: Optional[Lit_change_type],
        score: Optional[int],
    ) -> None:
-
        assert a_rawpath is None or isinstance(a_rawpath, bytes)
        assert b_rawpath is None or isinstance(b_rawpath, bytes)
        self.a_rawpath = a_rawpath

diff --git a/git/exc.py b/git/exc.py
@@ -139,7 +139,6 @@ def __init__(
        valid_files: Sequence[PathLike],
        failed_reasons: List[str],
    ) -> None:
-
        Exception.__init__(self, message)
        self.failed_files = failed_files
        self.failed_reasons = failed_reasons
@@ -170,7 +169,6 @@ def __init__(
        stderr: Union[bytes, str, None] = None,
        stdout: Union[bytes, str, None] = None,
    ) -> None:
-
        super(HookExecutionError, self).__init__(command, status, stderr, stdout)
        self._msg = "Hook('%s') failed%s"


diff --git a/git/index/base.py b/git/index/base.py
@@ -656,7 +656,7 @@ def _store_path(self, filepath: PathLike, fprogress: Callable) -> BaseIndexEntry
    def _entries_for_paths(
        self,
        paths: List[str],
-        path_rewriter: Callable,
        fprogress: Callable,
        entries: List[BaseIndexEntry],
    ) -> List[BaseIndexEntry]:

diff --git a/git/index/fun.py b/git/index/fun.py
@@ -76,7 +76,7 @@ def hook_path(name: str, git_dir: PathLike) -> str:
    return osp.join(git_dir, "hooks", name)


-def _has_file_extension(path):
    return osp.splitext(path)[1]


@@ -102,7 +102,7 @@ def run_commit_hook(name: str, index: "IndexFile", *args: str) -> None:
            relative_hp = Path(hp).relative_to(index.repo.working_dir).as_posix()
            cmd = ["bash.exe", relative_hp]

-        cmd = subprocess.Popen(
            cmd + list(args),
            env=env,
            stdout=subprocess.PIPE,
@@ -116,13 +116,13 @@ def run_commit_hook(name: str, index: "IndexFile", *args: str) -> None:
    else:
        stdout_list: List[str] = []
        stderr_list: List[str] = []
-        handle_process_output(cmd, stdout_list.append, stderr_list.append, finalize_process)
        stdout = "".join(stdout_list)
        stderr = "".join(stderr_list)
-        if cmd.returncode != 0:
            stdout = force_text(stdout, defenc)
            stderr = force_text(stderr, defenc)
-            raise HookExecutionError(hp, cmd.returncode, stderr, stdout)
    # end handle return code


@@ -394,7 +394,6 @@ def aggressive_tree_merge(odb: "GitCmdObjectDB", tree_shas: Sequence[bytes]) ->
                        out.append(_tree_entry_to_baseindexentry(theirs, 0))
                    # END handle modification
                else:
-
                    if ours[0] != base[0] or ours[1] != base[1]:
                        # they deleted it, we changed it, conflict
                        out.append(_tree_entry_to_baseindexentry(base, 1))

diff --git a/git/objects/commit.py b/git/objects/commit.py
@@ -345,9 +345,7 @@ def trailers(self) -> Dict[str, str]:
            Dictionary containing whitespace stripped trailer information.
            Only contains the latest instance of each trailer key.
        """
-        return {
-            k: v[0] for k, v in self.trailers_dict.items()
-        }

    @property
    def trailers_list(self) -> List[Tuple[str, str]]:
@@ -460,7 +458,7 @@ def _iter_from_process_or_stream(cls, repo: "Repo", proc_or_stream: Union[Popen,
            if proc_or_stream.stdout is not None:
                stream = proc_or_stream.stdout
        elif hasattr(proc_or_stream, "readline"):
-            proc_or_stream = cast(IO, proc_or_stream)
            stream = proc_or_stream

        readline = stream.readline

diff --git a/git/objects/fun.py b/git/objects/fun.py
@@ -190,7 +190,6 @@ def traverse_trees_recursive(
    # is a tree. If the match is a non-tree item, put it into the result.
    # Processed items will be set None
    for ti, tree_data in enumerate(trees_data):
-
        for ii, item in enumerate(tree_data):
            if not item:
                continue

diff --git a/git/objects/util.py b/git/objects/util.py
@@ -143,11 +143,11 @@ def utctz_to_altz(utctz: str) -> int:
    :param utctz: git utc timezone string, i.e. +0200
    """
    int_utctz = int(utctz)
-    seconds = ((abs(int_utctz) // 100) * 3600 + (abs(int_utctz) % 100) * 60)
    return seconds if int_utctz < 0 else -seconds


-def altz_to_utctz_str(altz: int) -> str:
    """Convert a timezone offset west of UTC in seconds into a git timezone offset string

    :param altz: timezone offset in seconds west of UTC

diff --git a/git/remote.py b/git/remote.py
@@ -826,7 +826,6 @@ def _get_fetch_info_from_stderr(
        progress: Union[Callable[..., Any], RemoteProgress, None],
        kill_after_timeout: Union[None, float] = None,
    ) -> IterableList["FetchInfo"]:
-
        progress = to_progress_instance(progress)

        # skip first line as it is some remote info we are not interested in

diff --git a/git/repo/base.py b/git/repo/base.py
@@ -498,7 +498,7 @@ def delete_head(self, *heads: "Union[str, Head]", **kwargs: Any) -> None:
    def create_tag(
        self,
        path: PathLike,
-        ref: Union[str, 'SymbolicReference'] = "HEAD",
        message: Optional[str] = None,
        force: bool = False,
        **kwargs: Any,
@@ -548,9 +548,8 @@ def _get_config_path(self, config_level: Lit_config_levels, git_dir: Optional[Pa
            else:
                return osp.normpath(osp.join(repo_dir, "config"))
        else:
-
-            assert_never(
-                config_level,  # type:ignore[unreachable]
                ValueError(f"Invalid configuration level: {config_level!r}"),
            )


diff --git a/git/util.py b/git/util.py
@@ -1049,7 +1049,7 @@ class IterableList(List[T_IterableObj]):

    __slots__ = ("_id_attr", "_prefix")

-    def __new__(cls, id_attr: str, prefix: str = "") -> "IterableList[IterableObj]":
        return super(IterableList, cls).__new__(cls)

    def __init__(self, id_attr: str, prefix: str = "") -> None:
@@ -1083,7 +1083,6 @@ def __getattr__(self, attr: str) -> T_IterableObj:
        return list.__getattribute__(self, attr)

    def __getitem__(self, index: Union[SupportsIndex, int, slice, str]) -> T_IterableObj:  # type: ignore
-
        assert isinstance(index, (int, str, slice)), "Index of IterableList should be an int or str"

        if isinstance(index, int):
@@ -1098,7 +1097,6 @@ def __getitem__(self, index: Union[SupportsIndex, int, slice, str]) -> T_Iterabl
        # END handle getattr

    def __delitem__(self, index: Union[SupportsIndex, int, slice, str]) -> None:
-
        assert isinstance(index, (int, str)), "Index of IterableList should be an int or str"

        delindex = cast(int, index)

diff --git a/pyproject.toml b/pyproject.toml
@@ -19,6 +19,7 @@ filterwarnings = 'ignore::DeprecationWarning'
# filterwarnings ignore::WarningType  # ignores those warnings

[tool.mypy]
disallow_untyped_defs = true
no_implicit_optional = true
warn_redundant_casts = true
@@ -29,6 +30,7 @@ implicit_reexport = true
# strict = true

# TODO: remove when 'gitdb' is fully annotated
[[tool.mypy.overrides]]
module = "gitdb.*"
ignore_missing_imports = true
@@ -43,3 +45,4 @@ omit = ["*/git/ext/*"]
[tool.black]
line-length = 120
target-version = ['py37']
Original file line number	Diff line number	Diff line change
Expand Up		@@ -110,6 +110,8 @@ To typecheck, run: `mypy -p git`

		To test, run: `pytest`

		Configuration for flake8 is in the ./.flake8 file.

		Configurations for mypy, pytest and coverage.py are in ./pyproject.toml.
Expand Down