updated setup_linter fmt lint targets

operator/hack/create_graph_openapi_schema.py

@@ -1,7 +1,8 @@
 import argparse
-import yaml
-import sys
 import copy
+import sys
+
+import yaml
 
 
 def getOpts(cmd_line_args):

operator/hack/csv_hack.py

@@ -1,7 +1,8 @@
-import yaml
 import argparse
 import sys
 
+import yaml
+
 
 def getOpts(cmd_line_args):
     parser = argparse.ArgumentParser(
@@ -20,7 +21,9 @@ def remove_versions(csv):
 
 
 def update_container_image(csv, version):
-    csv["metadata"]["annotations"]["containerImage"] = "seldonio/seldon-core-operator:"+version
+    csv["metadata"]["annotations"]["containerImage"] = (
+        "seldonio/seldon-core-operator:" + version
+    )
     return csv
 
 

operator/helm/split_resources.py

@@ -1,6 +1,7 @@
 import argparse
 import glob
 import re
+
 import yaml
 
 parser = argparse.ArgumentParser()

python/Makefile

@@ -95,17 +95,23 @@ push_conda:
 	@echo "Alternatively use anaconda upload to publish on own channel"
 
 setup_linter:
-	pip install black==20.8b1
+	pip install black==20.8b1 isort==5.7.0
 
 fmt:
 	black \
 		./ ../testing ../operator/helm ../operator/hack \
-		--exclude "(testing/scripts/proto|seldon_core/proto/|.eggs|.tox)"
+		--exclude "(testing|scripts|proto|seldon_core/proto/|.eggs|.tox)"
+	isort \
+		./ ../testing ../operator/helm ../operator/hack \
+		--skip testing --skip scripts --skip proto --skip seldon_core/proto --skip .eggs --skip .tox
 
 lint: licenses
 	black \
 		--check ./ ../testing \
 		--exclude "(testing/scripts/proto|seldon_core/proto/|.eggs|.tox)"
+	isort \
+		--check-only ./ ../testing \
+		--skip testing --skip scripts --skip proto --skip seldon_core/proto --skip .eggs --skip .tox
 	flake8
 	# Check if licenses have changed
 	git \

python/setup.py

@@ -1,5 +1,6 @@
 import os
 from itertools import chain
+
 from setuptools import find_packages, setup
 
 # Extra dependencies, with special 'all' key