From be59d1eb0588c05d38cd7cda0144f68dd09fb999 Mon Sep 17 00:00:00 2001
From: Matthias Mair <code@mjmair.com>
Date: Wed, 19 Jul 2023 13:50:26 +0200
Subject: [PATCH] P UI: Rearrange CI (#5276)

* rearrange CI

* small name change
---
 .github/workflows/qc_checks.yaml | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/.github/workflows/qc_checks.yaml b/.github/workflows/qc_checks.yaml
index d20d1afbb8..792ec1ca85 100644
--- a/.github/workflows/qc_checks.yaml
+++ b/.github/workflows/qc_checks.yaml
@@ -64,10 +64,10 @@ jobs:
         run: flake8 InvenTree --extend-ignore=D
 
   javascript:
-    name: Style [JS]
+    name: Style - Classic UI [JS]
     runs-on: ubuntu-20.04
 
-    needs: pep_style
+    needs: ['pep_style', 'pre-commit']
 
     steps:
       - uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8 # pin@v3.1.0
@@ -88,8 +88,8 @@ jobs:
   pre-commit:
     name: Style [pre-commit]
     runs-on: ubuntu-20.04
-
-    needs: pep_style
+    needs: paths-filter
+    if: needs.paths-filter.outputs.server == 'true' || needs.paths-filter.outputs.frontend == 'true'
 
     steps:
       - uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8 # pin@v3.1.0
@@ -187,7 +187,7 @@ jobs:
     name: Tests - DB [SQLite] + Coverage
     runs-on: ubuntu-20.04
 
-    needs: [ 'javascript', 'pre-commit' ]
+    needs: [ 'pep_style', 'pre-commit' ]
     continue-on-error: true # continue if a step fails so that coverage gets pushed
 
     env:
@@ -220,7 +220,7 @@ jobs:
   postgres:
     name: Tests - DB [PostgreSQL]
     runs-on: ubuntu-20.04
-    needs: [ 'javascript', 'pre-commit' ]
+    needs: [ 'pep_style', 'pre-commit' ]
 
     env:
       INVENTREE_DB_ENGINE: django.db.backends.postgresql
@@ -264,7 +264,7 @@ jobs:
     name: Tests - DB [MySQL]
     runs-on: ubuntu-20.04
 
-    needs: [ 'javascript', 'pre-commit' ]
+    needs: [ 'pep_style', 'pre-commit' ]
     if: github.event_name == 'push'
 
     env:
@@ -306,7 +306,7 @@ jobs:
         uses: ./.github/actions/migration
 
   migration-tests:
-    name: Run Migration Unit Tests
+    name: Tests - Migrations [PostgreSQL]
     runs-on: ubuntu-latest
     needs: paths-filter
     if: github.ref == 'refs/heads/master' && needs.paths-filter.outputs.migrations == 'true'
@@ -343,7 +343,7 @@ jobs:
         run: invoke test --migrations --report
 
   migrations-checks:
-    name: Run Database Migrations
+    name: Tests - Full Migration [SQLite]
     runs-on: ubuntu-latest
     needs: paths-filter
     if: github.ref == 'refs/heads/master' && needs.paths-filter.outputs.migrations == 'true'
@@ -391,11 +391,11 @@ jobs:
         chmod +rw /home/runner/work/InvenTree/db.sqlite3
         invoke migrate
 
-  plattform_ui:
-    name: Tests - UI Platform
+  platform_ui:
+    name: Tests - Platform UI
     runs-on: ubuntu-20.04
     timeout-minutes: 60
-    needs: paths-filter
+    needs: ['pre-commit', 'paths-filter']
     if: needs.paths-filter.outputs.frontend == 'true'
     env:
       INVENTREE_DB_ENGINE: sqlite3