ci: add ESLint check as blocking step before tests

- Add lint.yml reusable workflow that runs yarn lint
- Update tests.yml to call lint workflow first
- All test jobs now depend on lint passing

Author: pawelangelow

.github/workflows/lint.yml

@@ -0,0 +1,18 @@
+name: Lint
+
+on:
+  workflow_call:
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    name: ESLint
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install dependencies
+        uses: ./.github/actions/install-all-build-libs
+
+      - name: Run ESLint
+        run: yarn lint
+

.github/workflows/tests.yml

@@ -73,13 +73,17 @@ jobs:
             desktop:
               - 'redisinsight/desktop/**'
 
+  lint:
+    uses: ./.github/workflows/lint.yml
+    secrets: inherit
+
   frontend-tests:
-    needs: changes
+    needs: [changes, lint]
     if: |
-      startsWith(github.ref_name, 'fe/') || 
-      startsWith(github.ref_name, 'fe-be/') || 
-      startsWith(github.ref_name, 'feature/') || 
-      startsWith(github.ref_name, 'bugfix/') || 
+      startsWith(github.ref_name, 'fe/') ||
+      startsWith(github.ref_name, 'fe-be/') ||
+      startsWith(github.ref_name, 'feature/') ||
+      startsWith(github.ref_name, 'bugfix/') ||
       startsWith(github.ref_name, 'ric/') ||
       contains(github.event.pull_request.labels.*.name, 'run-all-tests') ||
       contains(github.event.pull_request.labels.*.name, 'run-frontend-tests')
@@ -95,12 +99,12 @@ jobs:
       type: unit
 
   backend-tests:
-    needs: changes
+    needs: [changes, lint]
     if: |
-      startsWith(github.ref_name, 'be/') || 
-      startsWith(github.ref_name, 'fe-be/') || 
-      startsWith(github.ref_name, 'feature/') || 
-      startsWith(github.ref_name, 'bugfix/') || 
+      startsWith(github.ref_name, 'be/') ||
+      startsWith(github.ref_name, 'fe-be/') ||
+      startsWith(github.ref_name, 'feature/') ||
+      startsWith(github.ref_name, 'bugfix/') ||
       startsWith(github.ref_name, 'ric/') ||
       contains(github.event.pull_request.labels.*.name, 'run-all-tests') ||
       contains(github.event.pull_request.labels.*.name, 'run-backend-tests')
@@ -116,12 +120,12 @@ jobs:
       type: unit
 
   integration-tests:
-    needs: changes
+    needs: [changes, lint]
     if: |
-      startsWith(github.ref_name, 'be/') || 
-      startsWith(github.ref_name, 'fe-be/') || 
-      startsWith(github.ref_name, 'feature/') || 
-      startsWith(github.ref_name, 'bugfix/') || 
+      startsWith(github.ref_name, 'be/') ||
+      startsWith(github.ref_name, 'fe-be/') ||
+      startsWith(github.ref_name, 'feature/') ||
+      startsWith(github.ref_name, 'bugfix/') ||
       startsWith(github.ref_name, 'ric/') ||
       contains(github.event.pull_request.labels.*.name, 'run-all-tests') ||
       contains(github.event.pull_request.labels.*.name, 'run-integration-tests')