Add linter isort to project and reformat imports.

Author: alfredeen

.github/workflows/ci.yaml

@@ -27,6 +27,16 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+
+      - name: Run isort  # pip-based (flexible)
+        run: |
+          pip install isort
+          isort . --check --diff --profile black
+
       - name: Run linter black
         uses: psf/black@stable
         with:

serve_event_listener/event_listener.py

@@ -6,15 +6,16 @@
 
 import requests
 import urllib3
-from urllib3.exceptions import HTTPError
 from kubernetes import client, config, watch
 from kubernetes.client.exceptions import ApiException
-from serve_event_listener.http_client import get as http_get, make_session
+from urllib3.exceptions import HTTPError
+
+from serve_event_listener.http_client import get as http_get
+from serve_event_listener.http_client import make_session
 from serve_event_listener.http_client import post as http_post
 from serve_event_listener.status_data import StatusData
 from serve_event_listener.status_queue import StatusQueue
 
-
 logger = logging.getLogger(__name__)
 
 # Disable urllib3 and Kubernetes client debug logs

serve_event_listener/http_client/client.py

@@ -2,7 +2,8 @@
 
 import logging
 import time
-from typing import Optional, Mapping, Any, Tuple, Callable
+from typing import Any, Callable, Mapping, Optional, Tuple
+
 import requests
 
 logger = logging.getLogger(__name__)

serve_event_listener/http_client/session.py

@@ -1,6 +1,7 @@
 """Configure requests.Session with retries, adapters, and TLS settings."""
 
 from typing import Mapping, Optional, Tuple
+
 import requests
 from requests.adapters import HTTPAdapter
 from urllib3.util.retry import Retry

serve_event_listener/main.py

@@ -1,7 +1,9 @@
 import argparse
 import logging
 import os
+
 from colorlog import ColoredFormatter
+
 from serve_event_listener.event_listener import EventListener
 
 # Configure the logger

serve_event_listener/status_queue.py

@@ -1,9 +1,11 @@
 import logging
 import queue
-import requests
 import threading
 import time
 from datetime import datetime, timezone
+
+import requests
+
 from serve_event_listener.http_client import post as http_post
 
 logger = logging.getLogger(__name__)
@@ -28,7 +30,8 @@ def __init__(
     def add(self, status_data) -> None:
         """Adds a status_data object to the queue."""
         logger.debug(
-            "K8s release status data added to queue. Queue now has length %s", self.queue.qsize() + 1
+            "K8s release status data added to queue. Queue now has length %s",
+            self.queue.qsize() + 1,
         )
         self.queue.put(status_data)
 

tests/http_client/test_http_helpers.py

@@ -1,9 +1,11 @@
 import unittest
 from unittest.mock import MagicMock
-from urllib3.util.retry import Retry
+
 import requests
 from requests.adapters import HTTPAdapter
-from serve_event_listener.http_client import make_session, _request
+from urllib3.util.retry import Retry
+
+from serve_event_listener.http_client import _request, make_session
 
 
 class TestMakeSession(unittest.TestCase):

tests/test_event_listener.py

@@ -1,7 +1,8 @@
 import logging
 import unittest
-from unittest.mock import patch, MagicMock
 from types import SimpleNamespace
+from unittest.mock import MagicMock, patch
+
 import requests
 
 from serve_event_listener.event_listener import EventListener

tests/test_status_queue.py

@@ -1,10 +1,11 @@
-import unittest
 import threading
 import time
-from unittest.mock import patch, MagicMock
+import unittest
 from types import SimpleNamespace
+from unittest.mock import MagicMock, patch
 
 import requests
+
 from serve_event_listener.status_queue import StatusQueue