LITE-25845 Add isort

* Add isort library and settings
* Remove flake8-import-order
* Run isort
* Fix build.yml
* Add isort run information to the README.md

Author: Hairash

.github/workflows/build.yml

@@ -15,7 +15,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: [3.7, 3.8]
+        python-version: ['3.7', '3.8']
     steps:
     - name: Checkout
       uses: actions/checkout@v2
@@ -73,7 +73,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: [3.10.1]
+        python-version: ['3.10']
     steps:
       - name: Checkout
         uses: actions/checkout@v2
@@ -105,10 +105,10 @@ jobs:
         uses: actions/checkout@v2
         with:
           fetch-depth: 0
-      - name: Set up Python '3.10.0'
+      - name: Set up Python '3.10'
         uses: actions/setup-python@v2
         with:
-          python-version: '3.10.0'
+          python-version: ['3.10']
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
@@ -145,10 +145,10 @@ jobs:
         uses: actions/checkout@v2
         with:
           fetch-depth: 0
-      - name: Set up Python '3.10.0'
+      - name: Set up Python '3.10'
         uses: actions/setup-python@v2
         with:
-          python-version: '3.10.0'
+          python-version: ['3.10']
       - name: Integration tests
         env:
           DOCKER_USERNAME: ${{ secrets.DOCKER_USERNAME }}

README.md

@@ -210,7 +210,10 @@ Development
 ===========
 
 1. Python 3.7 +
-0. Install dependencies `requirements/dev.txt`
+2. Install dependencies `requirements/dev.txt`
+3. We use `isort` library to order and format our imports, and we check it using `flake8-isort` library (automatically on `flake8` run).  
+For convenience you may run `poetry isort .` to order imports.
+
 
 Testing
 =======

dj_cqrs/__init__.py

@@ -2,5 +2,6 @@
 
 import django  # pragma: no cover
 
+
 if django.VERSION < (3, 2):  # pragma: no cover
     default_app_config = 'dj_cqrs.apps.CQRSConfig'

dj_cqrs/_validation.py

@@ -3,6 +3,8 @@
 import logging
 from inspect import getfullargspec, isfunction
 
+from django.utils.module_loading import import_string
+
 from dj_cqrs.constants import (
     DEFAULT_MASTER_AUTO_UPDATE_FIELDS,
     DEFAULT_MASTER_MESSAGE_TTL,
@@ -13,8 +15,6 @@
 from dj_cqrs.registries import MasterRegistry, ReplicaRegistry
 from dj_cqrs.transport import BaseTransport
 
-from django.utils.module_loading import import_string
-
 
 logger = logging.getLogger('django-cqrs')
 

dj_cqrs/apps.py

@@ -1,10 +1,10 @@
 #  Copyright © 2021 Ingram Micro Inc. All rights reserved.
 
-from dj_cqrs._validation import validate_settings
-
 from django.apps import AppConfig
 from django.conf import settings
 
+from dj_cqrs._validation import validate_settings
+
 
 class CQRSConfig(AppConfig):
     name = 'dj_cqrs'

dj_cqrs/controller/consumer.py

@@ -4,11 +4,11 @@
 import logging
 from contextlib import ExitStack
 
+from django.db import Error, close_old_connections, transaction
+
 from dj_cqrs.constants import SignalType
 from dj_cqrs.registries import ReplicaRegistry
 
-from django.db import Error, close_old_connections, transaction
-
 
 logger = logging.getLogger('django-cqrs')
 

dj_cqrs/dataclasses.py

@@ -1,12 +1,11 @@
 #  Copyright © 2022 Ingram Micro Inc. All rights reserved.
 
 from dateutil.parser import parse as dateutil_parse
+from django.utils import timezone
 
 from dj_cqrs.correlation import get_correlation_id
 from dj_cqrs.utils import get_json_valid_value, get_message_expiration_dt
 
-from django.utils import timezone
-
 
 class TransportPayload:
     """Transport message payload.

dj_cqrs/management/commands/cqrs_bulk_dump.py

@@ -5,12 +5,11 @@
 import sys
 import time
 
-from dj_cqrs.management.utils import batch_qs
-from dj_cqrs.registries import MasterRegistry
-
+import ujson
 from django.core.management.base import BaseCommand, CommandError
 
-import ujson
+from dj_cqrs.management.utils import batch_qs
+from dj_cqrs.registries import MasterRegistry
 
 
 class Command(BaseCommand):

dj_cqrs/management/commands/cqrs_bulk_load.py

@@ -3,12 +3,11 @@
 import os
 import sys
 
-from dj_cqrs.registries import ReplicaRegistry
-
+import ujson
 from django.core.management.base import BaseCommand, CommandError
 from django.db import DatabaseError, transaction
 
-import ujson
+from dj_cqrs.registries import ReplicaRegistry
 
 
 class Command(BaseCommand):

dj_cqrs/management/commands/cqrs_consume.py

@@ -2,11 +2,11 @@
 import multiprocessing
 import signal
 
+from django.core.management.base import BaseCommand, CommandError
+
 from dj_cqrs.registries import ReplicaRegistry
 from dj_cqrs.transport import current_transport
 
-from django.core.management.base import BaseCommand, CommandError
-
 
 class WorkersManager: