Use the new format session to re-format imports

Author: barisione

CHANGELOG.md

@@ -17,6 +17,7 @@ Internal changes
 - Refactored the code to parse MI records to decrease the number of regex matches to perform
 - Added `__all__` to all modules, which means that star imports (like `from pygdbmi.gdbmiparser import *`) will not pollute the namespace with modules used by pygdbmi itself
 - Added `nox -s format` to re-format the source code using the correct options
+- Reformatted all imports with `isort`, and use it as part of `nox -s lint` and `nox -s format`
 
 ## 0.10.0.2
 

example.py

@@ -3,12 +3,14 @@
 """
 Run with `python -m example`
 """
-import subprocess
 import os
+import subprocess
 import sys
-from pygdbmi.gdbcontroller import GdbController
 from distutils.spawn import find_executable
 
+from pygdbmi.gdbcontroller import GdbController
+
+
 SAMPLE_C_CODE_DIR = os.path.join(
     os.path.dirname(os.path.realpath(__file__)), "tests", "sample_c_app"
 )

noxfile.py

@@ -1,7 +1,9 @@
 import itertools
-import nox  # type: ignore
-from pathlib import Path
 import shutil
+from pathlib import Path
+
+import nox  # type: ignore
+
 
 nox.options.sessions = ["tests", "lint", "docs"]
 nox.options.reuse_existing_virtualenvs = True

pygdbmi/IoManager.py

@@ -3,24 +3,26 @@
 or pty.
 """
 import io
+import logging
+import os
 import select
 import time
 from pprint import pformat
-from typing import Union, List, Optional, Dict, Any, Tuple
+from typing import Any, Dict, List, Optional, Tuple, Union
+
 from pygdbmi import gdbmiparser
-import os
-import logging
 from pygdbmi.constants import (
     DEFAULT_GDB_TIMEOUT_SEC,
     DEFAULT_TIME_TO_CHECK_FOR_ADDITIONAL_OUTPUT_SEC,
     USING_WINDOWS,
     GdbTimeoutError,
 )
 
+
 if USING_WINDOWS:
     import msvcrt
-    from ctypes import windll, byref, wintypes, WinError, POINTER  # type: ignore
-    from ctypes.wintypes import HANDLE, DWORD, BOOL
+    from ctypes import POINTER, WinError, byref, windll, wintypes  # type: ignore
+    from ctypes.wintypes import BOOL, DWORD, HANDLE
 else:
     import fcntl
 

pygdbmi/gdbcontroller.py

@@ -6,12 +6,13 @@
 import logging
 import subprocess
 from distutils.spawn import find_executable
-from typing import Union, List, Optional
-from pygdbmi.IoManager import IoManager
+from typing import List, Optional, Union
+
 from pygdbmi.constants import (
     DEFAULT_GDB_TIMEOUT_SEC,
     DEFAULT_TIME_TO_CHECK_FOR_ADDITIONAL_OUTPUT_SEC,
 )
+from pygdbmi.IoManager import IoManager
 
 
 __all__ = ["GdbController"]

pygdbmi/gdbmiparser.py

@@ -12,9 +12,9 @@
 import re
 from typing import Callable, Dict, List, Match, Optional, Pattern, Tuple, Union
 
+from pygdbmi.gdbescapes import unescape
 from pygdbmi.printcolor import fmt_green
 from pygdbmi.StringStream import StringStream
-from pygdbmi.gdbescapes import unescape
 
 
 __all__ = [

setup.py

@@ -7,6 +7,7 @@
 
 from setuptools import find_packages, setup  # type: ignore
 
+
 EXCLUDE_FROM_PACKAGES = ["tests"]
 CURDIR = os.path.abspath(os.path.dirname(__file__))
 README = io.open("README.md", "r", encoding="utf-8").read()

tests/__main__.py

@@ -1,3 +1,4 @@
-from . import test_pygdbmi, static_tests
+from . import static_tests, test_pygdbmi
+
 
 exit(test_pygdbmi.main() + static_tests.main())

tests/test_pygdbmi.py

@@ -8,16 +8,17 @@
 
 import os
 import random
-import unittest
 import subprocess
-from time import time
+import unittest
 from distutils.spawn import find_executable
 from pprint import pprint
-from pygdbmi.StringStream import StringStream
+from time import time
+
+from pygdbmi.constants import USING_WINDOWS, GdbTimeoutError
+from pygdbmi.gdbcontroller import GdbController
 from pygdbmi.gdbescapes import advance_past_string_with_gdb_escapes, unescape
 from pygdbmi.gdbmiparser import parse_response
-from pygdbmi.gdbcontroller import GdbController
-from pygdbmi.constants import GdbTimeoutError, USING_WINDOWS
+from pygdbmi.StringStream import StringStream
 
 
 if USING_WINDOWS: