Pick a consistent type for __all__ (for now, list)

This makes all __all__ everywhere in the git package lists. Before,
roughly half were lists and half were tuples. There are reasonable
theoretical arguments for both, and in practice all tools today
support both. Traditionally using a list is far more common, and it
remains at least somewhat more common. Furthermore, git/util.py
uses a list and is currently written to append an element to it
that is conditionally defined on Windows (though it would probably
be fine for that to be the only list, since it reflects an actual
relevant difference about it).

The goal here is just to remove inconsistency that does not signify
anything and is the result of drift over time. If a reason (even
preference) arises to make them all tuples in the future, then that
is also probably fine.

Author: EliahKagan

git/cmd.py

@@ -5,7 +5,7 @@
 
 from __future__ import annotations
 
-__all__ = ("Git",)
+__all__ = ["Git"]
 
 import contextlib
 import io

git/config.py

@@ -5,7 +5,7 @@
 
 """Parser for reading and writing configuration files."""
 
-__all__ = ("GitConfigParser", "SectionConstraint")
+__all__ = ["GitConfigParser", "SectionConstraint"]
 
 import abc
 import configparser as cp

git/db.py

@@ -3,7 +3,7 @@
 
 """Module with our own gitdb implementation - it uses the git command."""
 
-__all__ = ("GitCmdObjectDB", "GitDB")
+__all__ = ["GitCmdObjectDB", "GitDB"]
 
 from gitdb.base import OInfo, OStream
 from gitdb.db import GitDB, LooseObjectDB

git/diff.py

@@ -3,7 +3,7 @@
 # This module is part of GitPython and is released under the
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
-__all__ = ("DiffConstants", "NULL_TREE", "INDEX", "Diffable", "DiffIndex", "Diff")
+__all__ = ["DiffConstants", "NULL_TREE", "INDEX", "Diffable", "DiffIndex", "Diff"]
 
 import enum
 import re

git/index/base.py

@@ -6,7 +6,7 @@
 """Module containing :class:`IndexFile`, an Index implementation facilitating all kinds
 of index manipulations such as querying and merging."""
 
-__all__ = ("IndexFile", "CheckoutError", "StageType")
+__all__ = ["IndexFile", "CheckoutError", "StageType"]
 
 import contextlib
 import datetime

git/index/fun.py

@@ -4,7 +4,7 @@
 """Standalone functions to accompany the index implementation and make it more
 versatile."""
 
-__all__ = (
+__all__ = [
     "write_cache",
     "read_cache",
     "write_tree_from_cache",
@@ -13,7 +13,7 @@
     "S_IFGITLINK",
     "run_commit_hook",
     "hook_path",
-)
+]
 
 from io import BytesIO
 import os

git/index/typ.py

@@ -3,7 +3,7 @@
 
 """Additional types used by the index."""
 
-__all__ = ("BlobFilter", "BaseIndexEntry", "IndexEntry", "StageType")
+__all__ = ["BlobFilter", "BaseIndexEntry", "IndexEntry", "StageType"]
 
 from binascii import b2a_hex
 from pathlib import Path

git/index/util.py

@@ -3,7 +3,7 @@
 
 """Index utilities."""
 
-__all__ = ("TemporaryFileSwap", "post_clear_cache", "default_index", "git_working_dir")
+__all__ = ["TemporaryFileSwap", "post_clear_cache", "default_index", "git_working_dir"]
 
 import contextlib
 from functools import wraps

git/objects/base.py

@@ -3,7 +3,7 @@
 # This module is part of GitPython and is released under the
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
-__all__ = ("Object", "IndexObject")
+__all__ = ["Object", "IndexObject"]
 
 import os.path as osp
 

git/objects/blob.py

@@ -3,7 +3,7 @@
 # This module is part of GitPython and is released under the
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
-__all__ = ("Blob",)
+__all__ = ["Blob"]
 
 from mimetypes import guess_type
 import sys

git/objects/commit.py

@@ -3,7 +3,7 @@
 # This module is part of GitPython and is released under the
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
-__all__ = ("Commit",)
+__all__ = ["Commit"]
 
 from collections import defaultdict
 import datetime

git/objects/fun.py

@@ -3,12 +3,12 @@
 
 """Functions that are supposed to be as fast as possible."""
 
-__all__ = (
+__all__ = [
     "tree_to_stream",
     "tree_entries_from_data",
     "traverse_trees_recursive",
     "traverse_tree_recursive",
-)
+]
 
 from stat import S_ISDIR
 

git/objects/submodule/util.py

@@ -1,13 +1,13 @@
 # This module is part of GitPython and is released under the
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
-__all__ = (
+__all__ = [
     "sm_section",
     "sm_name",
     "mkhead",
     "find_first_remote_branch",
     "SubmoduleConfigParser",
-)
+]
 
 from io import BytesIO
 import weakref

git/objects/tag.py

@@ -9,7 +9,7 @@
 For lightweight tags, see the :mod:`git.refs.tag` module.
 """
 
-__all__ = ("TagObject",)
+__all__ = ["TagObject"]
 
 import sys
 

git/objects/tree.py

@@ -3,7 +3,7 @@
 # This module is part of GitPython and is released under the
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
-__all__ = ("TreeModifier", "Tree")
+__all__ = ["TreeModifier", "Tree"]
 
 import sys
 

git/objects/util.py

@@ -5,7 +5,7 @@
 
 """Utility functions for working with git objects."""
 
-__all__ = (
+__all__ = [
     "get_object_type_by_name",
     "parse_date",
     "parse_actor_and_date",
@@ -17,7 +17,7 @@
     "Actor",
     "tzoffset",
     "utc",
-)
+]
 
 from abc import ABC, abstractmethod
 import calendar

git/remote.py

@@ -5,7 +5,7 @@
 
 """Module implementing a remote object allowing easy access to git remotes."""
 
-__all__ = ("RemoteProgress", "PushInfo", "FetchInfo", "Remote")
+__all__ = ["RemoteProgress", "PushInfo", "FetchInfo", "Remote"]
 
 import contextlib
 import logging

git/repo/base.py

@@ -5,7 +5,7 @@
 
 from __future__ import annotations
 
-__all__ = ("Repo",)
+__all__ = ["Repo"]
 
 import gc
 import logging

git/repo/fun.py

@@ -5,7 +5,7 @@
 
 from __future__ import annotations
 
-__all__ = (
+__all__ = [
     "rev_parse",
     "is_git_dir",
     "touch",
@@ -15,7 +15,7 @@
     "deref_tag",
     "to_commit",
     "find_worktree_git_dir",
-)
+]
 
 import os
 import os.path as osp