Improve order of imports and __all__ in git.objects.*

But not yet in git.objects.submodule.* modules.

Author: EliahKagan

Diff for: git/objects/submodule/base.py

@@ -1,6 +1,8 @@
 # This module is part of GitPython and is released under the
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
+__all__ = ["Submodule", "UpdateProgress"]
+
 import gc
 from io import BytesIO
 import logging
@@ -68,8 +70,6 @@
 
 # -----------------------------------------------------------------------------
 
-__all__ = ["Submodule", "UpdateProgress"]
-
 _logger = logging.getLogger(__name__)
 
 

Diff for: git/objects/submodule/root.py

@@ -1,10 +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__ = ["RootModule", "RootUpdateProgress"]
+
 import logging
 
 import git
 from git.exc import InvalidGitRepositoryError
+
 from .base import Submodule, UpdateProgress
 from .util import find_first_remote_branch
 
@@ -20,8 +23,6 @@
 
 # ----------------------------------------------------------------------------
 
-__all__ = ["RootModule", "RootUpdateProgress"]
-
 _logger = logging.getLogger(__name__)
 
 

Diff for: git/objects/submodule/util.py

@@ -1,12 +1,20 @@
 # This module is part of GitPython and is released under the
 # 3-Clause BSD License: https://opensource.org/license/bsd-3-clause/
 
-import git
-from git.exc import InvalidGitRepositoryError
-from git.config import GitConfigParser
+__all__ = (
+    "sm_section",
+    "sm_name",
+    "mkhead",
+    "find_first_remote_branch",
+    "SubmoduleConfigParser",
+)
+
 from io import BytesIO
 import weakref
 
+import git
+from git.config import GitConfigParser
+from git.exc import InvalidGitRepositoryError
 
 # typing -----------------------------------------------------------------------
 
@@ -22,15 +30,6 @@
     from git import Remote
     from git.refs import RemoteReference
 
-
-__all__ = (
-    "sm_section",
-    "sm_name",
-    "mkhead",
-    "find_first_remote_branch",
-    "SubmoduleConfigParser",
-)
-
 # { Utilities
 
 
@@ -65,7 +64,6 @@ def find_first_remote_branch(remotes: Sequence["Remote"], branch_name: str) -> "
 
 # } END utilities
 
-
 # { Classes