meta: Added sql linting to CI.

Author: adrianprelipcean

.github/workflows/cicd.yml

@@ -5,8 +5,8 @@ on:
   pull_request:
 
 jobs:
-  ci:
-    name: Continuous Integration
+  ci-node:
+    name: Continuous Integration Node
     runs-on: ubuntu-latest
     strategy:
       matrix:
@@ -20,10 +20,39 @@ jobs:
       - run: npm ci
       - run: npm test
       - run: npm run lint
+  ci-db:
+    name: Continuous Integration Database
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ["3.10"]
+    steps:
+    - uses: actions/checkout@v3
+      with:
+        fetch-depth: 0
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v3
+      with:
+        python-version: ${{ matrix.python-version }}
+    - name: Cache requirements
+      uses: actions/cache@v3
+      with:
+        path: ~/.cache/pip
+        key: ${{ runner.os }}-pip-${{ hashFiles('**/requirements.txt') }}
+        restore-keys: |
+          ${{ runner.os }}-pip-
+    - name: Install requirements
+      if: steps.cache.outputs.cache-hit != 'true'
+      run: |
+        python -m pip install --upgrade pip
+        pip install -r db/linting/requirements.txt
+    - name: Analysing the SQL code
+      run: |
+        bash db/linting/lint.sh 
   cd:
     name: Continuous Deployment
     runs-on: ubuntu-latest
-    needs: ci
+    needs: [ci-node,ci-db]
     if: startsWith(github.ref, 'refs/tags/v')
     steps:
       - name: Checkout the repository

.gitignore

@@ -2,4 +2,6 @@ node_modules/
 
 local_folder/
 
-dist/
+dist/
+
+.venv/

db/linting/.sqlfluff

@@ -0,0 +1,4 @@
+[sqlfluff]
+large_file_skip_char_limit = 0
+large_file_skip_byte_limit = 0
+exclude_rules = L016,LT05,RF05

db/linting/lint.sh

@@ -0,0 +1,2 @@
+echo 'Linting migrations folder'
+sqlfluff lint source/migrations/ --dialect postgres --ignore parsing --config db/linting/.sqlfluff 

db/linting/requirements.txt

@@ -0,0 +1 @@
+sqlfluff>=2.0.0, <=2.1.0