Add python code checks to CI

citusdata · Feb 10, 2023 · fb17ab7 · fb17ab7
1 parent baa56bb
commit fb17ab7
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -6,7 +6,7 @@ orbs:
 parameters:
   image_suffix:
     type: string
-    default: '-v7e4468f'
+    default: '-v9a494cd'
   pg13_version:
     type: string
     default: '13.9'
@@ -157,8 +157,17 @@ jobs:
     steps:
       - checkout
       - run:
-          name: 'Check Style'
+          name: 'Check C Style'
           command: citus_indent --check
+      - run:
+          name: 'Check Python style'
+          command: black --check .
+      - run:
+          name: 'Check Python import order'
+          command: isort --check .
+      - run:
+          name: 'Check Python lints'
+          command: flake8 .
       - run:
           name: 'Fix whitespace'
           command: ci/editorconfig.sh && git diff --exit-code

diff --git a/.flake8 b/.flake8
@@ -1,6 +1,6 @@
 [flake8]
-# Whitespace before ':' (E203) is ignored for black
-# Module level import not at top of file (E402) is ignored because of the way we do relative imports
+# E203 is ignored for black
+# E402 is ignored because of te way we do relative imports
 extend-ignore = E203, E402
 # black will truncate to 88 characters usually, but long string literals it
 # might keep. That's fine in most cases unless it gets really excessive.